Skip to content

Configure AppSec in CI #438

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Configure AppSec in CI #438

wants to merge 7 commits into from

Conversation

smlx
Copy link
Member

@smlx smlx commented May 28, 2025

Partially addresses #333

@smlx smlx force-pushed the appsec-ci branch 2 times, most recently from a0fdb19 to 6d9e309 Compare May 29, 2025 04:33
@smlx smlx marked this pull request as ready for review May 29, 2025 04:48
@smlx smlx requested a review from shreddedbacon May 29, 2025 04:48
@smlx
Copy link
Member Author

smlx commented May 29, 2025

I think this is good to go. There is some complexity in the four build workflows but hopefully the change is correct. The PR docker image build worked, so that is a good sign :D

@shreddedbacon
Copy link
Member

shreddedbacon commented Jun 5, 2025

@tobybellwood looping you in for some of the changes made to the build workflows as I'm not fully across these

@smlx
Copy link
Member Author

smlx commented Jun 6, 2025

Disabled the dockerhub attestation and rebased.

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.

2 participants