Skip to content

fix: ensure openapi spec demonstrates auth when cql2 filters apply#135

Merged
alukach merged 1 commit intomainfrom
fix/openapi-spec-cql2-filters
Feb 20, 2026
Merged

fix: ensure openapi spec demonstrates auth when cql2 filters apply#135
alukach merged 1 commit intomainfrom
fix/openapi-spec-cql2-filters

Conversation

@alukach
Copy link
Copy Markdown
Member

@alukach alukach commented Feb 20, 2026

Currently, the Swagger UI won't send auth credentials to endpoints if those endpoints are not marked as having a security scheme. While an endpoint may be public, the auth proxy might make use of those auth credentials to generate & apply CQL2 filters for row-level auth. As such, we should annotate an auth requirement to any endpoint that has CQL2 filters associated with it. It's important to note that these annotations do not actually denote an enforcement of auth, Swagger UI will still permit unauthenticated access. These annotations merely inform the Swagger UI client that auth credentials should be provided if possible.

@github-actions github-actions bot added the fix label Feb 20, 2026
@alukach alukach merged commit 7310cee into main Feb 20, 2026
4 checks passed
@alukach alukach deleted the fix/openapi-spec-cql2-filters branch February 20, 2026 18:37
alukach pushed a commit that referenced this pull request Feb 21, 2026
🤖 I have created a release *beep* *boop*
---


##
[1.0.1](v1.0.0...v1.0.1)
(2026-02-21)


### Bug Fixes

* ensure openapi spec demonstrates auth when cql2 filters apply
([#135](#135))
([7310cee](7310cee))
* support filtering bulk item creation
([2c4a791](2c4a791))


### Documentation

* **record-level-auth:** add filter factory guidance
([47c4e68](47c4e68))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: ds-release-bot[bot] <116609932+ds-release-bot[bot]@users.noreply.github.com>
This was referenced Mar 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant