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

Add ability to disable network in the container while building. #186

Merged
merged 1 commit into from
Apr 15, 2023

Conversation

Bluebugs
Copy link
Contributor

Description:

This can only be used with vendored directory. It does provide two main benefit. First not setting up network, means that the build will not be able to get any data from the outside. It give you that guarantee that only the source code that is in your vendor directory was used. Second it is faster to setup a container without network.

Checklist:

  • Lint and formatter run with no errors.
  • Tests all pass.

Where applicable:

  • Public APIs match existing style.

This can only be used with vendored directory. It does provide two main
benefit. First not setting up network, means that the build will not be
able to get any data from the outside. It give you that guarantee that
only the source code that is in your vendor directory was used. Second
it is faster to setup a container without network.
@Bluebugs Bluebugs requested a review from lucor April 14, 2023 21:37
@Bluebugs
Copy link
Contributor Author

@Bluebugs
Copy link
Contributor Author

@Jacalz enjoy :-)

Copy link
Member

@Jacalz Jacalz left a comment

Choose a reason for hiding this comment

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

Nice 👍

@Bluebugs Bluebugs merged commit 7070259 into master Apr 15, 2023
@Bluebugs Bluebugs deleted the features/no-network branch April 15, 2023 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants