Skip to content

Add tests for Auth handling of _blob endpoint redirects#803

Merged
bgunebakan merged 1 commit into
mainfrom
fix/800-_blob-endpoint-redirects-in-cratedb-loose-basic-http-auth-headers
May 18, 2026
Merged

Add tests for Auth handling of _blob endpoint redirects#803
bgunebakan merged 1 commit into
mainfrom
fix/800-_blob-endpoint-redirects-in-cratedb-loose-basic-http-auth-headers

Conversation

@bgunebakan
Copy link
Copy Markdown
Contributor

Summary of the changes / Why this is an improvement

Add new parametrized tests across all four blob operations (blob_exists, blob_put, blob_del, blob_get) to increase test coverage on _blob redirects.

Checklist

@bgunebakan bgunebakan self-assigned this May 15, 2026
@bgunebakan bgunebakan requested review from amotl, kneth and matriv May 15, 2026 14:15
Copy link
Copy Markdown
Member

@amotl amotl left a comment

Choose a reason for hiding this comment

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

Thank you. Please let @matriv also acknlowedge the patch, because he knows about relevant details better than me.

Copy link
Copy Markdown
Contributor

@matriv matriv left a comment

Choose a reason for hiding this comment

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

thank you!

@bgunebakan bgunebakan merged commit de3c5b9 into main May 18, 2026
17 checks passed
@bgunebakan bgunebakan deleted the fix/800-_blob-endpoint-redirects-in-cratedb-loose-basic-http-auth-headers branch May 18, 2026 07:29
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.

_blob endpoint redirects in CrateDB loose Basic HTTP Auth headers

3 participants