fix: restrict webhook secret access to admin-only

[riderx]

Summary (AI generated)

• Restricted webhook secret reads to admin users in supabase/migrations/260224000000_fix_webhooks_select_permission.sql via an updated RLS policy.
• Updated permissions so CI policy checks for webhook access no longer fail while keeping admin workflows unchanged.

Motivation (AI generated)

Webhook secrets must remain admin-only. The previous policy allowed broader access than intended, and CI surfaced failures around permission coverage and security expectations.

Business Impact (AI generated)

This closes a security gap by preventing non-admin users from reading webhook secrets, reducing risk without changing normal app behavior for authorized administrators.

Test Plan (AI generated)

• Run bun lint
• Confirmed branch diff against origin/main contains only the intended migration change
• Run/observe related backend CI checks in this PR run

Generated with AI

[coderabbitai]

Warning

Rate limit exceeded

@riderx has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 26 minutes and 57 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between ca23f62 and 09fcec4.

📒 Files selected for processing (1)

• supabase/migrations/20260224000000_fix_webhooks_select_permission.sql

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch riderx/fix-ci-issues

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 09fcec4506

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Keep webhook list readable for non-admin org members

Changing the SELECT policy to require 'admin' removes read access for all non-admin members, but the current frontend still treats webhook viewing as broadly readable: Webhooks.vue fetches on mount for all users and only gates mutating actions behind canManageWebhooks, and webhooksStore.fetchWebhooks() uses direct from('webhooks').select('*') reads (src/pages/settings/organization/Webhooks.vue and src/stores/webhooks.ts). In practice, read-only org members now get empty/error states instead of webhook visibility, which is a functional regression beyond the stated goal of protecting secrets.

Useful? React with 👍 / 👎.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud