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

[Feature]: Export sessions #28

Open
1 task done
jonbarrow opened this issue Jun 1, 2024 · 0 comments
Open
1 task done

[Feature]: Export sessions #28

jonbarrow opened this issue Jun 1, 2024 · 0 comments
Labels
approved The topic is approved by a developer feature A feature request

Comments

@jonbarrow
Copy link
Member

Checked Existing

  • I have checked the repository for duplicate issues.

What feature do you want to see added?

Add the ability to export sessions, with all packets decrypted/decoded already.

We already convert packets to JSON when sending them to the renderer, so maybe we can just dump the whole session as a JSON list of connections? As like a .nexj file (so that we can use the file select window to target them specifically)? The files would be kind of big but we'd also be able to view the data in any text editor.

Alternatively we could dump to pcap/pcapng dumps instead, much like how https://github.com/PretendoNetwork/HokakuCTR does. But we'd need to make sure we identify those dumps uniquely, and we don't yet have a pcap/pcapng writer (though one could be made). Though with this method you'd only be able to view the dumps in our viewer, and maybe Wireshark?

Why do you want to have this feature?

Currently it's difficult to debug certain issues since they rely on dumps which we cannot give to outside contributors. This is due to them being encrypted using the users NEX password, and we can't freely hand that data out. This was specifically mentioned by @wolfendale on Discord, and I agree this is an issue.

I've brought up want to add session exporting since the beginning of this viewer, but only really in the context of speed increases for parsing. But it can also help with safely giving session data to other parties without the need for NEX credentials.

Any other details to share? (OPTIONAL)

Unless #26 is merged, any changes related to this issue should target the rewrite branch.

@jonbarrow jonbarrow added awaiting-approval Topic has not been approved or denied feature A feature request labels Jun 1, 2024
@DaniElectra DaniElectra added approved The topic is approved by a developer and removed awaiting-approval Topic has not been approved or denied labels Jun 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved The topic is approved by a developer feature A feature request
Projects
Status: Todo
Development

No branches or pull requests

2 participants