Skip to content

(docs): Update SBP with SSRF protection note#2196

Open
localden wants to merge 17 commits intomainfrom
localden/ssrf
Open

(docs): Update SBP with SSRF protection note#2196
localden wants to merge 17 commits intomainfrom
localden/ssrf

Conversation

@localden
Copy link
Contributor

@localden localden commented Feb 3, 2026

No description provided.

🏠 Remote-Dev: homespace
🏠 Remote-Dev: homespace
jenn-newton
jenn-newton previously approved these changes Feb 3, 2026
…file move

- Update 4 relative links in elicitation.mdx to new canonical path
- Add CODEOWNERS entry for new file location so auth-wg still reviews
@pcarleton
Copy link
Member

Pushed a fix for a couple things the file move missed:

  • 4 relative links in elicitation.mdx still pointed to the old ../basic/security_best_practices path
  • CODEOWNERS glob didn't cover the new file location, so auth-wg wouldn't get tagged on future edits

pcarleton
pcarleton previously approved these changes Feb 5, 2026
Copy link
Member

@pcarleton pcarleton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM to me overall, one nit on link local

Co-authored-by: Paul Carleton <paulcarletonjr@gmail.com>
Co-authored-by: Paul Carleton <paulcarletonjr@gmail.com>
🏠 Remote-Dev: homespace
🏠 Remote-Dev: homespace
🏠 Remote-Dev: homespace
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation security

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants