Skip to content

Conversation

Luigi31415
Copy link
Contributor

Description

This PR adds a new parser for handling the output of the wg show all dump command from WireGuard. The parser converts the command output into structured JSON, making it easier for users to integrate WireGuard data into their workflows for monitoring or automation.

Summary of Changes

  • Added a parser that handles the wg show all dump command output.
  • The parser produces structured JSON with detailed device and peer information, including fields such as device, privateKey, publicKey, listenPort, peers, and more.

Request for Collaboration

  • Test Cases: I would appreciate help with generating comprehensive test cases to cover possible variations of the wg command output. This will ensure that the parser handles all edge cases effectively.
  • Review Feedback: Any feedback on the schema and implementation would be very helpful for refinement.

Closes #605

Copy link
Owner

@kellyjonbrazil kellyjonbrazil left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution!

@Luigi31415
Copy link
Contributor Author

Hi @kellyjonbrazil,

Thank you for reviewing this PR! I wanted to note that the output of wg is consistent across most operating systems. Based on this, I believe we can confidently proceed with merging this PR by including tests for Ubuntu and Windows, as they should be sufficient to cover the majority of use cases.

Please let me know if there’s anything else I can assist with to help finalize this.

Best regards,
Luigi

@Luigi31415 Luigi31415 marked this pull request as ready for review November 24, 2024 11:44
@kellyjonbrazil
Copy link
Owner

Nice, thanks! I'll merge and this will make it int the next release. Thanks!

@kellyjonbrazil kellyjonbrazil merged commit 7fbe1e9 into kellyjonbrazil:dev Nov 24, 2024
21 checks passed
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.

3 participants