-
Notifications
You must be signed in to change notification settings - Fork 75
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
ci: bump actions/download-artifact version #3291
Conversation
PR ChecklistHow to use this checklistHow to use this checklistPR AuthorFor each section, check a box when it is true. PR ReviewerCheck that the PR checklist action did not fail. Bug ReferencesNone. Confirm
How to properly reference fixed bugs
Test UpdatesUnit Tests
Integration Tests
Documentation
Does this PR require review from someone outside the core ubuntu-pro-client team?
|
7cd92bc
to
7d5f941
Compare
So, CI ran an the new version of the action is not pulling our artifact. Needs some investigation on why. |
@paride I see there is an upload-artifact action too - maybe there is where the mismatch happens. This gotta be simpler than expected I guess |
7d5f941
to
19be20a
Compare
This mitigates CVE-2024-42471 Signed-off-by: Renan Rodrigo <[email protected]>
19be20a
to
339cb5d
Compare
Solved - this is now good to go |
Thanks. Sorry with all the things around .5 I didn't get to this earlier. |
@paride don't worry - it was easier than expected and at least we didn't waste your time (: |
Why is this needed?
This PR solves all of our problems because it mitigates CVE-2024-42471.
Dependabot told us about this one, but I'm reopening here to let CI run and make sure all is good.
See #3289 for context.
Test Steps
Just let the CI run