Skip to content

fix: set correct valid methods env var - #4818

Merged
sweatybridge merged 1 commit into
developfrom
revert-4781-sweatybridge-patch-3
Feb 7, 2026
Merged

fix: set correct valid methods env var#4818
sweatybridge merged 1 commit into
developfrom
revert-4781-sweatybridge-patch-3

Conversation

@sweatybridge

Copy link
Copy Markdown
Contributor

Reverts #4781

@sweatybridge
sweatybridge requested a review from a team as a code ownerFebruary 7, 2026 12:54
@sweatybridgesweatybridge changed the title Revert "fix: set correct valid methods env var"fix: set correct valid methods env varFeb 7, 2026
@coveralls

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 21780415311

Details

  • 1 of 1(100.0%) changed or added relevant line in 1 file are covered.
  • 7 unchanged lines in 2 files lost coverage.
  • Overall coverage remained the same at 55.376%

Files with Coverage ReductionNew Missed Lines%
internal/storage/rm/rm.go280.61%
internal/gen/keys/keys.go512.9%
TotalsCoverage Status
Change from base Build 21779169557:0.0%
Covered Lines:7123
Relevant Lines:12863

💛 - Coveralls

@sweatybridge
sweatybridge merged commit eb7f8cc into developFeb 7, 2026
10 of 11 checks passed
@sweatybridge
sweatybridge deleted the revert-4781-sweatybridge-patch-3 branch February 7, 2026 13:05
@singh-inder

Copy link
Copy Markdown
Contributor

Hi @sweatybridge, just wanted to flag that this env var hasn’t been updated in auth v2.186.0 yet. Related: #4820

@sweatybridge

Copy link
Copy Markdown
ContributorAuthor

lol that's unfortunate. Let me revert the revert. Would you like to close the gotrue PR so it doesn't confuse more folks?

@sweatybridge

Copy link
Copy Markdown
ContributorAuthor

For reference #4817

mandric pushed a commit to mandric/sitemgr that referenced this pull request Mar 22, 2026
CLI versions before 2.76.4 have a broken ES256 JWT signing bug
(supabase/cli#4818) that causes GoTrue to reject service role keys,
leading to cryptic integration test failures. Add an upfront version
check in both local-dev.sh and test-integration.sh so developers get
a clear error message before anything starts.
https://claude.ai/code/session_012opDAZ8NGx22St9Knf1kVE
Sign up for freeto 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.

3 participants

@sweatybridge@coveralls@singh-inder