Skip to content

[PM-43699] Fix up gov showing when we're not ready for it yet - #8825

Open
JaredScar wants to merge 1 commit into
mainfrom
ac/pm-43699-gov-server-option-in-swagger-public-api
Open

JaredScar wants to merge 1 commit into
mainfrom
ac/pm-43699-gov-server-option-in-swagger-public-api

Conversation

@JaredScar

Copy link
Copy Markdown
Contributor

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-43699

📔 Objective

Rids of the gov bitwarden in swagger API docs until we are ready for it.

📸 Screenshots

image

@JaredScar JaredScar added ai-review Request a Claude code review t:bugfix Change Type - Bugfix labels Nov 6, 2026
@JaredScar
JaredScar requested a review from eliykat November 6, 2026 18:36
@github-actions

github-actions Bot commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR filters CloudRegion.Gov out of the cloud-region loop in AddSwaggerGen, so the published Public API swagger spec no longer lists the Gov server or its OAuth2 security definition. Other CloudRegionConfig.All consumers (allowed domains and SSO callback URIs in Constants.cs) are unaffected. The new xUnit tests check that the Gov server and its Gov_server security scheme are absent, and that the US and EU entries are still present.

Code Review Details

No findings.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review Request a Claude code review t:bugfix Change Type - Bugfix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants