Skip to content

Enforce fetch APM protection#485

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

Enforce fetch APM protection#485
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