Skip to content

Conversation

@JPeer264
Copy link
Member

@JPeer264 JPeer264 commented Dec 2, 2025

📜 Description

We had an issue in getsentry/sentry-javascript#18372 that it created an issue, even though an issue has added.

Screenshot 2025-12-02 at 12 00 56

The issue was that the URL was given instead of the short link such as #100 (btw, I think this example will also be taken as false positive)

💡 Motivation and Context

Since we don't have a shared action right now, this is the only option.

💚 How did you test it?

We tested it in our repo 👍, also there is a regex101.com link as reference

📝 Checklist

You have to check all boxes before merging:

  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.

@philprime philprime added the ready-to-merge Use this label to trigger all PR workflows label Dec 2, 2025
@philprime philprime enabled auto-merge (squash) December 2, 2025 12:07
@philprime philprime merged commit 03afb63 into main Dec 2, 2025
165 of 176 checks passed
@philprime philprime deleted the jp/allow-url branch December 2, 2025 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Use this label to trigger all PR workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants