Skip to content

release-26.2: ui: move TimezoneProvider to be inside RequiredLogin#169786

Draft
kyle-a-wong wants to merge 1 commit intocockroachdb:release-26.2from
kyle-a-wong:backport26.2-169751
Draft

release-26.2: ui: move TimezoneProvider to be inside RequiredLogin#169786
kyle-a-wong wants to merge 1 commit intocockroachdb:release-26.2from
kyle-a-wong:backport26.2-169751

Conversation

@kyle-a-wong
Copy link
Copy Markdown
Contributor

Backport 1/1 commits from #169751 on behalf of @kyle-a-wong.

/cc @cockroachdb/release


Fixes a bug where the login page was making a request for cluster settings in order to create the timezone provider context. This request results in a 401 unauthorized since users need to be authorized to request cluster setting data. In version before 26.3, this bug made it impossible to login as a non default tenant via OIDC login, since this unauthorized response caused tenant cookies to be cleared before user could click the OIDC button.

Epic: None
Release note: None


Release justification: Fix for a bug that prevents non-default tenant OIDC login

Fixes a bug where the login page was making a request for
cluster settings in order to create the timezone provider
context. This request results in a 401 unauthorized since
users need to be authorized to request cluster setting
data. In version before 26.3, this bug made it impossible
to login as a non default tenant via OIDC login, since
this unauthorized response caused tenant cookies to
be cleared before user could click the OIDC button.

Epic: None
Release note: None
@blathers-crl
Copy link
Copy Markdown

blathers-crl Bot commented May 5, 2026

Thanks for opening a backport.

Before merging, please confirm that the change does not break backwards compatibility and otherwise complies with the backport policy. Include a brief release justification in the PR description explaining why the backport is appropriate. All backports must be reviewed by the TL for the owning area. While the stricter LTS policy does not yet apply, please exercise judgment and consider gating non-critical changes behind a disabled-by-default feature flag when appropriate.

@trunk-io
Copy link
Copy Markdown
Contributor

trunk-io Bot commented May 5, 2026

Merging to release-26.2 in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

@blathers-crl blathers-crl Bot added backport Label PR's that are backports to older release branches T-observability labels May 5, 2026
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@kyle-a-wong
Copy link
Copy Markdown
Contributor Author

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

Labels

backport Label PR's that are backports to older release branches T-observability

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants