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

chore: update of deps#26

Open
mboronin wants to merge 1 commit into
mobilejazz:masterfrom
mboronin:bump-version
Open

chore: update of deps#26
mboronin wants to merge 1 commit into
mobilejazz:masterfrom
mboronin:bump-version

Conversation

@mboronin

@mboronin mboronin commented Feb 7, 2023

Copy link
Copy Markdown

Suggesting to update the dependencies to avoid vulnerabilities

  • Asana task link:
    Merge / Pull request information:

@jgimenez jgimenez left a comment

Copy link
Copy Markdown
Member

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 :)

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

RUN apt-get -yqq remove \

Copy link
Copy Markdown
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

mboronin commented Feb 8, 2023

Copy link
Copy Markdown
Author

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