Skip to content

Handle non-OK ConfigResponseMessages gracefully in Python.#368

Merged
adamshapiro0 merged 3 commits intomasterfrom
bad-config-response
Apr 17, 2025
Merged

Handle non-OK ConfigResponseMessages gracefully in Python.#368
adamshapiro0 merged 3 commits intomasterfrom
bad-config-response

Conversation

@adamshapiro0
Copy link
Collaborator

Fixes

  • Check the response type and data size before decoding the config response payload

@adamshapiro0 adamshapiro0 requested a review from axlan April 17, 2025 12:44
@adamshapiro0 adamshapiro0 self-assigned this Apr 17, 2025
Copy link
Collaborator

@axlan axlan left a comment

Choose a reason for hiding this comment

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

LGTM

@adamshapiro0 adamshapiro0 merged commit 91fdd7b into master Apr 17, 2025
14 checks passed
@adamshapiro0 adamshapiro0 deleted the bad-config-response branch April 17, 2025 19:54
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