Skip to content
This repository has been archived by the owner on Mar 23, 2023. It is now read-only.

chore: update of deps #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

mboronin
Copy link

@mboronin mboronin commented Feb 7, 2023

Suggesting to update the dependencies to avoid vulnerabilities

  • Asana task link:
    Merge / Pull request information:

Copy link
Member

@jgimenez jgimenez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for your contribution! I added just a simple comment but other than that, I think it's great to have this image updated :)

&& apt-get -yqq remove \
&& rm -rf "/opt/src/libreswan-${SWAN_VER}"

RUN apt-get -yqq remove \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Separating the cleanup phase into a different step makes the image larger. I would keep it in the same RUN statement.

@mboronin
Copy link
Author

mboronin commented Feb 8, 2023

I'm still testing it actually, sorry for opening a PR too early. Trying to make the image a bit more slim, since there seem to be quite an increase in size after the upgrade

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants