-
Notifications
You must be signed in to change notification settings - Fork 8
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
Förvaltning av gamla beroenden #18
Comments
Tjena, tack för info! Tanken är att det ska göras uppdateringar i repot med senare versioner av beroenden. Kikar på det och återkopplar 👍 |
Super! |
@fredriknordlander hur går det, vi använder er fork o security larmar hög)? |
Tjena, runnit lite vatten sedan denna rapporterades, men vi ska planera in och åtgärda under ngn av kommande sprintar. |
@fredriknordlander tjena, när fixas det har ni tidsram? |
This comment was marked as off-topic.
This comment was marked as off-topic.
@JessicaPry so how is it with this issue? Who will fix it if Fredrik is gone? |
This comment was marked as off-topic.
This comment was marked as off-topic.
@tomppa-p @matthiaspalmer @jenniferskoglund someone, please, reply how is it going? |
This comment was marked as off-topic.
This comment was marked as off-topic.
@jonassodergren i ask you to help to contact someone. |
This comment was marked as off-topic.
This comment was marked as off-topic.
@halge - sorry för radiotystnaden. Tyvärr finns det ingen spikad tidsram ännu. Teamet som ska åtgärda dessa delar har ej kommit på plats, så det har inte gått att planera in arbetet. Dock är det topp prio att åtgärda dessa sec.issues. |
This comment was marked as off-topic.
This comment was marked as off-topic.
Hejsan, jag fick frågan av Fredrik och kollade närmare på denna. SCA-verktyget (dependabot) kunde inte hitta just det felet och beroendet som pekas ut i denna. Jag kikade om beroendet återfinns inte heller i GitHubs dependecygraph för projektet, ej i den heller. Provade också SCA-dependabot-inställningarna i en tom fork av projektet för att se om det var något knas med själva inställningarna för projektet. Inget funnet. Kan det vara en äldre version än den i main som skannats av Xray? Jag körde också sidokoll med senaste NVD-informationen utanför GitHub's verktyg och då hittade OWASPs SCA-verktyg https://jeremylong.github.io/DependencyCheck/dependency-check-maven/ några criticals som inte GitHubs egna verktyg fann. Alla dess criticals verkar lösa sig om man uppgraderar spring boot-versionen. Här ser vi att man kan få lite olika resultat beroende på vilket verktyg man använder helt enkelt också. Så det här är eg två frågor : finns de specifika CVE's som pekas ut jiran kvar i senaste main, och är critical ? Nej inte vad githubs verktyg eller OWASP säger. Så här är mina förslag:
|
desutom docker build . falleras (man kan bara använda :latest tag fri #26):
|
@halge |
Jag har nu lyckats uppgradera spring version till 3.2.0 och smoke test typ fungerar. Alla spring CRITICALS är borta. Snakeyaml också gick att uppgradera. Enda som är kvar är det felet som finns i issuen. Som Josef har också skrivit, det är lite lurigt. OWASP verktyget har spårat den till hibernate-core. Jag har kollat för nyare versioner men hittade ingen på själva som finns i projekt, har provat med en liknade men felet finns kvar. Den används inte direkt från kod. Utan den tillgängliggör json types till hibernate om den lyckas laddas innan eller samtidigt som hibernate. Jag tror att det är därför den är svårt hittad. Nu har jag pratat med Jonas och vi har kommit överens att jag committar till två olika feature branches. En med uppdaterat spring samt snakeyaml och andra där hinernate-core är också borta. Vi kör acceptans test och gå darifrån. Uppdaterar issue igen när jag är klar med committen. |
Uppgraderat spring boot mm: 18-förvaltning-av-gamla-beroenden |
@janderssonse @ayeshabhatti78 inform when those changes are in main, please. |
@janderssonse @ayeshabhatti78 when main branch will be updated? what is the plan? |
Hej! @halge Jag har själv inga mergemandat för detta projekt, men ska kolla med med min kollega m.fl vad status är på dessa alt. om jag skulle kunna få mergemandat för enklare fixar som säkerhet o liknande. |
Hej. @halge Vi ska börja acceptanstesta de fixes vi gjort för denna issue och några till. Jobbet drar igång till veckan. Efter detta, förutsatt att det fungerar som tänkt, så kommer vi att merga in detta på main. |
Hej igen! Hur är status på det här nu? När tror ni att ni kommer kunna merga in detta på main? |
What happened?
Beroenden som används i applikationen är utdaterade och gör att scanningsprogram som Xray rapporterar sårbarheter som har dykt upp i de beroendena. Ex. så används version 1.61 av Dom4j som har de kända sårbarheterna CVE-2020-10683 och CVE-2018-1000632. Finns det någon plan för hur förvaltning av mjukvaran ska göras? Kommer det göras uppdateringar i repot med senare versioner av beroenden?
Steps To Reproduce
Sårbarheterna borde rapporteras vid scanning med Xray eller liknande analysprogram.
What did you expect?
Vi skulle önska senare versioner av de beroenden som används.
Version
No response
Optional Screenshots
No response
Relevant log output
No response
The text was updated successfully, but these errors were encountered: