Skip to content

feat(COD-7048): use new caching mechanism for docker image#266

Draft
baltoiteodor wants to merge 3 commits into
mainfrom
teodor/COD-7048
Draft

feat(COD-7048): use new caching mechanism for docker image#266
baltoiteodor wants to merge 3 commits into
mainfrom
teodor/COD-7048

Conversation

@baltoiteodor
Copy link
Copy Markdown
Contributor

@baltoiteodor baltoiteodor commented Jun 4, 2026

tested here: https://github.com/lacework-dev/WebGoat/actions/runs/26957051714/job/79541710114

All runs resulted in the same cache key: 812f42254c72ade044350b0d2025e540ad8768b2da304a4f7198934512980b04

Comment thread src/util.ts Outdated
}
}

export async function generateCacheKey(): Promise<string | undefined> {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It might be easier to reuse the function runCodesec but with an extra parameter compteCacheKey in order to make sure that we using the exact same scan parameters when calculating the cache key and when running the scans.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sure, sounds good

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