Skip to content

chore: Fix incorrect SDK log classification#188

Merged
gasperzgonec merged 6 commits intomainfrom
gasperz/ISS-289723
Apr 20, 2026
Merged

chore: Fix incorrect SDK log classification#188
gasperzgonec merged 6 commits intomainfrom
gasperz/ISS-289723

Conversation

@gasperzgonec
Copy link
Copy Markdown
Contributor

Changed locations that ran user code and reported as SDK logs.

Description

Currently some of the logs have been tagged as SDK logs in the logs that have been produced by the user code.
These points should be updated to always log as user logs instead.

Connected Issues

Checklist

  • Tests added/updated and ran with npm run test OR no tests needed.
  • Ran backwards compatibility tests with npm run test:backwards-compatibility.
  • Code formatted and checked with npm run lint.
  • Tested airdrop-template linked to this PR.
  • Documentation updated and provided a link to PR / new docs OR no docs needed.

@github-actions
Copy link
Copy Markdown

⚠️ Heads-up: This repository will be blocked from any work other than patching.

File src/common/control-protocol.ts is not allowed to be modified in this patch.
The following vulnerability issues are past SLA:

Note that there is significant latency in updating this list. Please reach out on #antifragile if you are in a hurry or have an emergency.

@github-actions
Copy link
Copy Markdown

⚠️ Heads-up: This repository will be blocked from any work other than patching.

File src/http/axios-client.ts is not allowed to be modified in this patch.
The following vulnerability issues are past SLA:

Note that there is significant latency in updating this list. Please reach out on #antifragile if you are in a hurry or have an emergency.

@github-actions
Copy link
Copy Markdown

⚠️ Heads-up: This repository will be blocked from any work other than patching.

File src/http/axios-client.ts is not allowed to be modified in this patch.
The following vulnerability issues are past SLA:

Note that there is significant latency in updating this list. Please reach out on #antifragile if you are in a hurry or have an emergency.

@github-actions
Copy link
Copy Markdown

⚠️ Heads-up: This repository will be blocked from any work other than patching.

File src/attachments-streaming/attachments-streaming-pool.test.ts is not allowed to be modified in this patch.
The following vulnerability issues are past SLA:

Note that there is significant latency in updating this list. Please reach out on #antifragile if you are in a hurry or have an emergency.

@gasperzgonec gasperzgonec marked this pull request as ready for review April 20, 2026 07:32
@gasperzgonec gasperzgonec requested review from a team and radovanjorgic as code owners April 20, 2026 07:32
@gasperzgonec gasperzgonec merged commit abd0364 into main Apr 20, 2026
8 of 9 checks passed
@gasperzgonec gasperzgonec deleted the gasperz/ISS-289723 branch April 20, 2026 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants