Skip to content

Support authStrategy=none with a local Admin session (quality-of-life DX improvement) - #104

Merged
rudokemper merged 1 commit into
mainfrom
rk/authstrategy-none
Aug 13, 2026
Merged

Support authStrategy=none with a local Admin session (quality-of-life DX improvement) #104
rudokemper merged 1 commit into
mainfrom
rk/authstrategy-none

Conversation

@rudokemper

Copy link
Copy Markdown
Member

Goal

To support the same DX improvement as ConservationMetrics/gc-explorer#559 did for GC Explorer.

Screenshots

n/a

What I changed and why

  • Similar approach to set server/middleware/local-auth.ts and bypass in middleware/oauth.global.ts
  • Here, we did have to do some additional work to replace an auth0Enabled boolean to match the Explorer convention of checking for authStrategy (auth0 | none).

How I convinced myself this is right

Trying locally.

What I'm not doing here

LLM use disclosure

Cursor Grok 4.6 did the implementation based on reviewing the code changes from ConservationMetrics/gc-explorer#559. Then I made changes, mostly to verbose comments.

@rudokemper rudokemper self-assigned this Aug 13, 2026

@conservationtimothy conservationtimothy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice to have the same QOL improvment here.

@rudokemper
rudokemper merged commit 6a2b236 into main Aug 13, 2026
1 check passed
@rudokemper
rudokemper deleted the rk/authstrategy-none branch August 13, 2026 12:54
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