Skip to content

fix issue leading to slowdown/unbounded memory growth in binary json processing#5

Open
tomqext wants to merge 2 commits intodevelopfrom
ERD-1923_ros1_fix_decode_binary_websocket_frames_before_json_parse
Open

fix issue leading to slowdown/unbounded memory growth in binary json processing#5
tomqext wants to merge 2 commits intodevelopfrom
ERD-1923_ros1_fix_decode_binary_websocket_frames_before_json_parse

Conversation

@tomqext
Copy link
Copy Markdown

@tomqext tomqext commented Mar 25, 2026

See upstream PR for more info RobotWebTools#1199

Note - our workflows are forked from a while ago and don't work. I tried updating them with upstreams but they fail too due to linting errors. Will have an internal conversation about aligning this repo with upstream - for now the tests will fail but they're old/not ours and this doesn't matter here

@tomqext tomqext force-pushed the ERD-1923_ros1_fix_decode_binary_websocket_frames_before_json_parse branch from 6b1d81d to 86cba2b Compare March 25, 2026 12:15
@tomqext tomqext changed the title Erd 1923 ros1 fix decode binary websocket frames before json parse fix issue leading to slowdown/unbounded memory growth in binary json processing Mar 25, 2026
Copy link
Copy Markdown
Collaborator

@nitish3693 nitish3693 left a comment

Choose a reason for hiding this comment

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

LGTM

@tomqext tomqext force-pushed the ERD-1923_ros1_fix_decode_binary_websocket_frames_before_json_parse branch from cd72e7a to 86cba2b Compare March 25, 2026 12:39
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