Skip to content

Conversation

@dehydr8
Copy link
Member

@dehydr8 dehydr8 commented Aug 30, 2025

This PR attempts to do the following:

  1. Merge master into wireshark-update-4.4.5 (that branch has a stable update to 4.4.5), fixing conflicts
  2. Fix broken IOGraph tests after minor API changes
  3. Formatting *.cpp files using clang-format

#19

dehydr8 and others added 6 commits June 23, 2024 13:04
There is some "smart" logic around what the data directory path should
be, but fails to return the correct directory. This change patches the
get_datafile_dir() function to return DATA_DIR.
This fixes the colorfilters file not loading correctly.
@dehydr8 dehydr8 force-pushed the wireshark-update-4.4.5-merge-master branch from cd5fd86 to 639b8e5 Compare August 30, 2025 19:05
@dehydr8
Copy link
Member Author

dehydr8 commented Aug 30, 2025

@dipeshwalia can you please try reproducing the issue in this one?

@dipeshwalia
Copy link
Contributor

@dipeshwalia can you please try reproducing the issue in this one?

tcp_pref_issue.pcap.zip, ill try with this branch as well.

const result = wg.lib.setPref("diameter", "tcp.port", '3871')

the above pref works with the master branch and not with 4.4.5 branch.

thank you!

@dipeshwalia
Copy link
Contributor

@dehydr8, it seems like we are good with this PR. thank you,

@dehydr8 dehydr8 added the enhancement New feature or request label Sep 1, 2025
@dehydr8 dehydr8 merged commit a2667c8 into master Sep 1, 2025
1 check passed
@github-actions
Copy link

github-actions bot commented Sep 1, 2025

🎉 This PR is included in version 1.9.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants