Skip to content

🛡️ Sentinel: [security improvement] Add bot protection and fix JSON-LD XSS#48

Draft
saynchowdhury wants to merge 1 commit into
mainfrom
sentinel/bot-protection-and-json-ld-xss-fix-9100617654429444180
Draft

🛡️ Sentinel: [security improvement] Add bot protection and fix JSON-LD XSS#48
saynchowdhury wants to merge 1 commit into
mainfrom
sentinel/bot-protection-and-json-ld-xss-fix-9100617654429444180

Conversation

@saynchowdhury

Copy link
Copy Markdown
Member

This pull request implements several security enhancements to protect the application from automated exploitation and XSS vulnerabilities.

🛡️ Security Improvements:

  1. Bot Protection (High Priority):

    • Added checkBotId() from the botid library to the voting and AI-filter API routes.
    • This prevents automated vote manipulation and protects the AI-filter endpoint from resource exhaustion and inflated LLM costs by ensuring requests originate from legitimate users.
  2. JSON-LD XSS Prevention (High Priority):

    • Identified a vulnerability where JSON.stringify was used directly within dangerouslySetInnerHTML for script tags.
    • Implemented a new safeJsonLd utility that stringifies data and escapes < and > characters (using \u003c and \u003e).
    • Applied this utility to all JSON-LD scripts in app/layout.tsx, app/opportunity/[id]/page.tsx, and blog pages.

✅ Verification:

  • Ran pnpm lint to ensure no syntax or type errors were introduced.
  • Verified that all modified routes and components are correctly importing and utilizing the new security logic.
  • Recorded a new learning in .jules/sentinel.md regarding safe JSON-LD handling.

PR created automatically by Jules for task 9100617654429444180 started by @saynchowdhury

…D XSS

- Added `checkBotId()` protection to `/api/opportunities/[id]/vote` and `/api/opportunities/ai-filter` to prevent automated abuse and resource exhaustion.
- Implemented `safeJsonLd` utility in `lib/utils.ts` to safely escape `<` and `>` in JSON-LD scripts.
- Replaced direct `JSON.stringify` with `safeJsonLd` in all JSON-LD script tags across the application to prevent XSS.

Co-authored-by: saynchowdhury <205922286+saynchowdhury@users.noreply.github.com>
@google-labs-jules

Copy link
Copy Markdown

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

@vercel

vercel Bot commented Jun 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
foundery.space Error Error Jun 28, 2026 8:59am

@supabase

supabase Bot commented Jun 28, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project tpvpacwoquygbykcjqle because there are no changes detected in /supabase/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@coderabbitai

coderabbitai Bot commented Jun 28, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 5417d486-99ab-4cf5-878d-cfbaee65999e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch sentinel/bot-protection-and-json-ld-xss-fix-9100617654429444180

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

Comment @coderabbitai help to get the list of available commands.

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.

1 participant