Skip to content

Conversation

hf
Copy link
Contributor

@hf hf commented Feb 28, 2025

Removes API key (anon, service_role) checks from Envoy as new projects no longer need these. They were superseded by origin protection keys and API key checks at API gateway.

@hf hf requested a review from a team as a code owner February 28, 2025 16:10
@samrose
Copy link
Collaborator

samrose commented Feb 28, 2025

security and functional testing for this PR is relying on what is discussed here https://github.com/supabase/infrastructure/pull/21834#issuecomment-2691098231

@hf hf force-pushed the hf/remove-api-key-checks-lds-supabase branch from 5b9b7b1 to a879985 Compare March 1, 2025 11:37
@hf hf force-pushed the hf/remove-api-key-checks-lds-supabase branch from a879985 to 675311c Compare March 1, 2025 12:28
@hf hf merged commit 79c663b into develop Mar 1, 2025
12 checks passed
@hf hf deleted the hf/remove-api-key-checks-lds-supabase branch March 1, 2025 13:47
damonrand pushed a commit to cepro/postgres that referenced this pull request Jun 15, 2025
* feat: remove api key checks in envoy

* bump vars

* add `x-sb-error-code` header to envoy lds.yaml when api key or opk don't match
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.

3 participants