fix(deletion): allow IRSA S3 credentials - #6495
Merged
Merged
Conversation
added 3 commits
August 21, 2026 12:43
Co-authored-by: coder 0 <d97ebdbb198c7237c94f84ea8bb8a73583ea067407eebd0062abbb3962527fb1@buzz.block.builderlab.xyz> Signed-off-by: coder 0 <d97ebdbb198c7237c94f84ea8bb8a73583ea067407eebd0062abbb3962527fb1@buzz.block.builderlab.xyz>
…credentials * origin/main: chore(release): release Buzz Desktop version 0.5.18 (#6489) Co-authored-by: coder 0 <d97ebdbb198c7237c94f84ea8bb8a73583ea067407eebd0062abbb3962527fb1@buzz.block.builderlab.xyz> Signed-off-by: coder 0 <d97ebdbb198c7237c94f84ea8bb8a73583ea067407eebd0062abbb3962527fb1@buzz.block.builderlab.xyz>
Co-authored-by: coder 0 <d97ebdbb198c7237c94f84ea8bb8a73583ea067407eebd0062abbb3962527fb1@buzz.block.builderlab.xyz> Signed-off-by: coder 0 <d97ebdbb198c7237c94f84ea8bb8a73583ea067407eebd0062abbb3962527fb1@buzz.block.builderlab.xyz>
bradseiler
marked this pull request as ready for review
August 21, 2026 22:03
kalvinnchau
approved these changes
Aug 21, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
buzz-admin deletionsruns inside bb-public relay pods, where S3 credentials are provided by the pod credential chain rather than staticBUZZ_S3_ACCESS_KEY/BUZZ_S3_SECRET_KEYvalues. The deletion CLI was requiring those two env vars to be non-empty before constructing the shared media storage client, so it could not reach the existing default AWS credential chain.What
MediaStorage::newcan useCredentials::default().Risk Assessment
Low and scoped to the operator-only community deletion CLI startup path. The shared media storage credential validation still owns static-vs-default credential selection and still rejects mixed partial credentials.
Testing
At committed head
0a86c2914b1f97caf4788a771048aa8d9d9d88acwith a clean worktree before and after (git rev-parse HEADbefore/after matched):just fmt-check— passed.cargo test -p buzz-deletion— passed: 12 passed, 9 ignored.cargo test -p buzz-media— passed: 120 passed;static_creds_round_trip_against_minioremained ignored because it requires live MinIO.cargo test -p buzz-admin— passed: 1 passed.cargo clippy -p buzz-deletion --all-targets -- -D warnings— passed.buzz-admin, then rantarget/debug/buzz-admin deletions drainwithBUZZ_S3_ACCESS_KEY=andBUZZ_S3_SECRET_KEY=' 'plusAWS_ACCESS_KEY_ID/AWS_SECRET_ACCESS_KEYfallback credentials; command exited0, proving startup transitions past deletion S3 key validation and exercises the shared default credential-chain branch using AWS env fallback credentials.git push origin HEAD:seiler/deletion-irsa-credentials— passed; pre-push hooks passed.Not run: the full
TESTING.mdlive-local relay workflow. Docker Desktop currently refuses CLI access on this machine withSign in to continue using Docker Desktop. Membership in the [squareup] organization is required.References
buzz://message?channel=9e4aabc6-414c-4978-aba7-b9f5228776de&id=177c5b8ad9e78a647f438ec7040d25f0c20a7c2b8678dd0e34768effe053f7f4Generated with Codex