Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[C++] Fix Wireshark dissector decode send command metadata behavior #13471

Conversation

Demogorgon314
Copy link
Member

Motivation

Currently, the Wireshark dissector doesn't decode send message command metadata correctly, and some TCP packages are marked incorrectly when using pulsar dissector.

Modifications

  • Fix decode send command metadata behavior
  • Fix dissect pulsar message marked incorrect TCP packages
  • Add DEBUG message for debugging
  • Support decode more Send command metadata info

Documentation

  • no-need-doc

@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Dec 23, 2021
@Demogorgon314
Copy link
Member Author

/pulsarbot run-failure-checks

@codelipenghui codelipenghui added this to the 2.10.0 milestone Dec 23, 2021
@codelipenghui codelipenghui merged commit ee823cf into apache:master Dec 23, 2021
@Demogorgon314 Demogorgon314 deleted the Fix_not_enough_data_to_dissect_command branch December 24, 2021 00:56
Technoboy- pushed a commit to Technoboy-/pulsar that referenced this pull request Dec 29, 2021
…pache#13471)

### Motivation
Currently, the Wireshark dissector doesn't decode send message command metadata correctly, and some TCP packages are marked incorrectly when using pulsar dissector.

### Modifications
- Fix decode send command metadata behavior
- Fix dissect pulsar message marked incorrect TCP packages
- Add DEBUG message for debugging
- Support decode more Send command metadata info
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-not-needed Your PR changes do not impact docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants