Skip to content

fix: skip bucket seeding#5158

Open
7ttp wants to merge 2 commits intosupabase:developfrom
7ttp:fix/storage-no-buckets
Open

fix: skip bucket seeding#5158
7ttp wants to merge 2 commits intosupabase:developfrom
7ttp:fix/storage-no-buckets

Conversation

@7ttp
Copy link
Copy Markdown
Contributor

@7ttp 7ttp commented May 1, 2026

TL;DR

skip local storage bucket seeding when no buckets are configured.
the unnecessary GET /storage/v1/bucket call was causing supabase start to fail with
Error status 400: {"statusCode":"403","error":"Unauthorized","message":"new row violates row-level security policy"}

sol:

buckets.Run now returns early for local runs when [storage.buckets] is empty, configured local buckets and linked project bucket seeding still use the existing path unchanged

added a test aswell

ref:

@7ttp 7ttp requested a review from a team as a code owner May 1, 2026 09:39
@7ttp 7ttp requested a review from avallete May 1, 2026 09:40
@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 25209902879

Coverage decreased (-0.01%) to 63.721%

Details

  • Coverage decreased (-0.01%) from the base build.
  • Patch coverage: 3 of 3 lines across 1 file are fully covered (100%).
  • 5 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

5 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
internal/utils/git.go 5 57.14%

Coverage Stats

Coverage Status
Relevant Lines: 15662
Covered Lines: 9980
Line Coverage: 63.72%
Coverage Strength: 7.02 hits per line

💛 - Coveralls

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.

supabase start fails supabase start fails with {"statusCode":"403","error":"Unauthorized","message":"new row violates row-level security policy"}

2 participants