Skip to content

Enforce fetch APM protection - #485

Merged
parfeon merged 2 commits into
masterfrom
refactor/fallback-fetch-by-default
Nov 20, 2025
Merged

parfeon merged 2 commits into
masterfrom
refactor/fallback-fetch-by-default

Conversation

@parfeon

@parfeon parfeon commented Nov 20, 2025

Copy link
Copy Markdown
Contributor

refactor(web-transport): enforce fetch APM protection

Enforce use of the fetch function from the context, which is not affected by APM monkey patching.

Enforce use of the `fetch` function from the context, which is not affected by APM monkey patching.
@parfeon parfeon self-assigned this Nov 20, 2025
@parfeon
parfeon requested a review from mohitpubnub as a code owner November 20, 2025 11:54
@parfeon parfeon added status: done This issue is considered resolved. priority: high This PR should be reviewed ASAP. type: refactor This PR contains refactored existing features. labels Nov 20, 2025
@pubnub-ops-terraform

pubnub-ops-terraform commented Nov 20, 2025 •

Copy link
Copy Markdown

✅ Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@mohitpubnub mohitpubnub left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@parfeon

parfeon commented Nov 20, 2025

Copy link
Copy Markdown
Contributor Author

@pubnub-release-bot release

@parfeon
parfeon merged commit 762e185 into master Nov 20, 2025
9 checks passed
@parfeon
parfeon deleted the refactor/fallback-fetch-by-default branch November 20, 2025 12:39
@pubnub-release-bot

Copy link
Copy Markdown
Contributor

🚀 Release successfully completed 🚀

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

Labels

priority: high This PR should be reviewed ASAP. status: done This issue is considered resolved. type: refactor This PR contains refactored existing features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants