Skip to content

Create SECURITY.md#1508

Open
Amanyelmahdey wants to merge 1 commit intoworkos:mainfrom
Amanyelmahdey:patch-1
Open

Create SECURITY.md#1508
Amanyelmahdey wants to merge 1 commit intoworkos:mainfrom
Amanyelmahdey:patch-1

Conversation

@Amanyelmahdey
Copy link

Description

Documentation

Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.

[ ] Yes

If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.

@Amanyelmahdey Amanyelmahdey requested a review from a team as a code owner February 28, 2026 14:15
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 28, 2026

Greptile Summary

This PR adds a SECURITY.md file to the repository, but it appears to be the unmodified GitHub template with placeholder content.

Critical Issues:

  • Placeholder instructional text remains on lines 5-6 and 17-21 ("Use this section to tell people...")
  • Version numbers (5.1.x, 5.0.x, 4.0.x) don't match the actual project version (8.7.0)
  • No actual vulnerability reporting process is documented

The file needs to be customized with WorkOS-specific information before merging.

Confidence Score: 1/5

  • This PR should not be merged as-is - it contains only placeholder content
  • The SECURITY.md file is completely uncustomized from the GitHub template, with incorrect version numbers and no actual security reporting instructions, making it misleading for users trying to report vulnerabilities
  • SECURITY.md requires complete rewrite with actual WorkOS security policy

Important Files Changed

Filename Overview
SECURITY.md Added GitHub SECURITY.md template without customization - contains placeholder text and incorrect version numbers

Last reviewed commit: efa4ca1

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, 3 comments

Edit Code Review Agent Settings | Greptile

Comment on lines +5 to +6
Use this section to tell people about which versions of your project are
currently being supported with security updates.
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove this placeholder instructional text

Suggested change
Use this section to tell people about which versions of your project are
currently being supported with security updates.

Comment on lines +10 to +13
| 5.1.x | :white_check_mark: |
| 5.0.x | :x: |
| 4.0.x | :white_check_mark: |
| < 4.0 | :x: |
Copy link
Contributor

Choose a reason for hiding this comment

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

Version numbers don't match the actual project version (currently 8.7.0 per package.json)

Comment on lines +17 to +21
Use this section to tell people how to report a vulnerability.

Tell them where to go, how often they can expect to get an update on a
reported vulnerability, what to expect if the vulnerability is accepted or
declined, etc.
Copy link
Contributor

Choose a reason for hiding this comment

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

Replace placeholder text with actual vulnerability reporting instructions (e.g., WorkOS security email, expected response time, etc.)

Suggested change
Use this section to tell people how to report a vulnerability.
Tell them where to go, how often they can expect to get an update on a
reported vulnerability, what to expect if the vulnerability is accepted or
declined, etc.
To report a security vulnerability, please email security@workos.com with a detailed description of the issue. Our security team will respond within 48 hours to acknowledge receipt and provide next steps.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant