Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Poistetaan curl frontend-imagesta #6009

Merged
merged 1 commit into from
Nov 22, 2024
Merged

Conversation

terolaakso
Copy link
Contributor

curlilla on riippuvuus openldap-kirjastoon, jossa on korjaamattomia haavoittuvuuksia. Poistetaan curl frontend imagesta ja lisätään se playwright-imageen, jossa sitä tarvitaan. playwright image on käytössä vain GitHubissa testejä ajettaessa, joten haavoittuvuudella ei ole siellä merkitystä.

@terolaakso terolaakso added the tech Tekninen muutos, esim. refaktorointi label Nov 20, 2024
@terolaakso terolaakso force-pushed the remove-curl-from-frontend branch from b0f4437 to 627f526 Compare November 22, 2024 08:55
This is to remove dependency to openldap and its vulnerabilities
@terolaakso terolaakso force-pushed the remove-curl-from-frontend branch from 627f526 to d8d3ee5 Compare November 22, 2024 10:29
@terolaakso terolaakso marked this pull request as ready for review November 22, 2024 11:27
@terolaakso
Copy link
Contributor Author

Tämän jälkeen tulee uusi PR, jossa wait-for-url.sh poistetaan frontend-hakemistosta. Sitä ei pystynyt poistamaan tässä samassa PR:ssä, koska e2e käyttää masterin versiota, jossa edelleen on vanha frontend-hakemistoon viittaava run-tests.sh

@terolaakso terolaakso merged commit 7a1bedd into master Nov 22, 2024
31 of 32 checks passed
@terolaakso terolaakso deleted the remove-curl-from-frontend branch November 22, 2024 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech Tekninen muutos, esim. refaktorointi
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants