Skip to content

fix: (azblob) Handle 404 responses gracefully#871

Closed
eplightning wants to merge 1 commit intobuchgr:masterfrom
eplightning:fix-azure-404-handling
Closed

fix: (azblob) Handle 404 responses gracefully#871
eplightning wants to merge 1 commit intobuchgr:masterfrom
eplightning:fix-azure-404-handling

Conversation

@eplightning
Copy link
Contributor

Similarly to how its already done in all other proxy backends, Azure Blob proxy backend needs to be able to handle case of missing blob in its Get method.

Without this, certain clients break and do not work over gRPC protocol. Example of one such a client is https://github.com/moonrepo/moon , not sure about bazel itself.

Copy link
Collaborator

@mostynb mostynb left a comment

Choose a reason for hiding this comment

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

lgtm- thanks (I don't have an azure account to test with)

@mostynb
Copy link
Collaborator

mostynb commented Nov 14, 2025

Landed this, with a slightly different commit message. Thanks again.

@mostynb mostynb closed this Nov 14, 2025
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