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

libcnb-test: Use --trust-extra-buildpacks with pack build #855

Merged
merged 1 commit into from
Aug 28, 2024

Conversation

edmorley
Copy link
Member

@edmorley edmorley commented Aug 27, 2024

To force the builder to be trusted after changes were made in Pack CLI v0.35.0 to improve security:
buildpacks/pack#2228

The new flag to restore the old behaviour was only added in Pack v0.35.1, so using libcnb-test now requires that Pack version or newer. (Adding a version check would require calling out to Pack before every test execution, which is not worth it given that it only affects tests, and this will be a one-off transition.)

Fixes #854.
GUS-W-16600037.

@edmorley edmorley self-assigned this Aug 27, 2024
@edmorley edmorley force-pushed the edmorley/trust-extra-buildpacks branch 2 times, most recently from 24b5c24 to ab15c23 Compare August 27, 2024 20:49
@edmorley edmorley marked this pull request as ready for review August 27, 2024 20:53
@edmorley edmorley requested a review from a team as a code owner August 27, 2024 20:53
@edmorley edmorley enabled auto-merge (squash) August 27, 2024 20:55
To force the builder to be trusted after changes were made in Pack
CLI v0.35.0 to improve security.

The new flag to restore the old behaviour was only added in Pack
v0.35.1, so using `libcnb-test` now requires that Pack version or
newer. (Adding a version check would require calling out to Pack
before every test execution, which is not worth it given that it only
affects tests, and this will be a one-off transition.)

Fixes #854.
GUS-W-16600037.
@edmorley edmorley force-pushed the edmorley/trust-extra-buildpacks branch from ab15c23 to e5a524e Compare August 27, 2024 21:55
@edmorley edmorley merged commit b49fe5c into main Aug 28, 2024
4 checks passed
@edmorley edmorley deleted the edmorley/trust-extra-buildpacks branch August 28, 2024 04:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

libcnb-test: Pass --trust-extra-buildpacks when running pack build
2 participants