Skip to content

fix: enable RLS for failed webhook events#175

Closed
saurabhhhcodes wants to merge 1 commit into
Coder-s-OG-s:mainfrom
saurabhhhcodes:codex/failed-webhook-events-rls-161
Closed

fix: enable RLS for failed webhook events#175
saurabhhhcodes wants to merge 1 commit into
Coder-s-OG-s:mainfrom
saurabhhhcodes:codex/failed-webhook-events-rls-161

Conversation

@saurabhhhcodes
Copy link
Copy Markdown
Contributor

Summary

  • enables Row Level Security on failed_webhook_events
  • intentionally adds no public policies so anon/authenticated clients are blocked by default
  • keeps service-role retry/dead-letter workflows working, since service role bypasses RLS

Validation

  • git diff --check
  • inspected existing migration and service-role-only usages in webhook retry/process flows

Fixes #161

For GSSoC scoring, please add/keep GSSOC26, level:intermediate, and type:security.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 19, 2026

Someone is attempting to deploy a commit to the codersogs-3057's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Copy Markdown

Hey @saurabhhhcodes

You have 4 open PRs right now. The limit is 3 at a time.

Please get your existing PRs merged or closed before opening new ones:

This PR will remain open but won't be reviewed until you're under the limit. See our Contributing Guidelines for details.

@saurabhhhcodes
Copy link
Copy Markdown
Contributor Author

Closing this in favor of #177, which carries the same RLS fix with a safer rerunnable migration plus a regression test. That should also bring me back within the repo open-PR limit so the newer PR can be reviewed cleanly. Thanks!

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.

failed_webhook_events table missing RLS policies

1 participant