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

Support indirect property length (and maybe more, from a fork) #130

Open
yirkha opened this issue Jun 9, 2024 · 3 comments
Open

Support indirect property length (and maybe more, from a fork) #130

yirkha opened this issue Jun 9, 2024 · 3 comments

Comments

@yirkha
Copy link

yirkha commented Jun 9, 2024

Hi, let me first thank you for making this crate, it allowed me to do what I needed to do. And it has mostly "just worked", which is always a pleasure!

But I hit some limitations as well, for example with the variable length "Address" properties of some events from the Microsoft.Windows.Winsock.AFD provider, because their length is stored indirectly in another property. And this is just a TODO in the code so far...

Luckily for me, I found there was a fork of ferrisetw (jxy-s) that already had this feature implemented, together with some extra serialization support and whatnot. So I just changed my Cargo.toml to reference that repo/branch instead of the published crate and carried on. Great.

But now I'd like to go further with my project and referencing dependencies from random Git repos is not really feasible, so I have a basic question for you, @n4r1b: How much time do you have for this project, would you be willing to work on adding/integrating this feature, release a new version at some point etc.?

Of course, usually it would be rather the author of the fork going back to the upstream with a PR. So a similar question to @jxy-s here, would you be interested in making some PRs to get your improvements merged upstream maybe?

And I know everybody's time is limited, so I could help with rebasing/cleaning/testing/publishing PRs too. I'd just like to discuss first before stepping on anybody's toes 🙂

Thank you

@jxy-s
Copy link
Contributor

jxy-s commented Jun 9, 2024

Feel free to cherry-pick any commits upstream if they're useful. Looks like my fork has diverged a bit, so there could be some conflicts to resolve.

@n4r1b
Copy link
Owner

n4r1b commented Jun 27, 2024

Hi @yirkha, as you can see from the time it took me to answer back unfortunately nowadays I don't have much time for this project, but I'll gladly help resolving conflicts and helping with PRs and testing. So if you want to do as @jxy-s suggested and cherry-pick the commits you need I can later take a look and help with solving problems in the PR :)

@yirkha
Copy link
Author

yirkha commented Jul 1, 2024

Okay, thanks for your blessing, @jxy-s, and no worries at all, @n4r1b, I think we all know how it goes - guess why I haven't touched any of this further myself either in the meantime ;-) I'll revisit Johnny's changes one of these days, try to rebase, test and make some PR(s)

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

No branches or pull requests

3 participants