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

Multiple fixes and features #18

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

rsrdesarrollo
Copy link

@rsrdesarrollo rsrdesarrollo commented Feb 29, 2024

Hi!

First, thanks for your plugin, but, to be honest, I felt like he was falling a bit behind compared to the original collaborator, so I've taken the liberty of making a few improvements.

Improve generate payload performance:

The main problem here was that the "generate a new payload" button created a new client each time, generating some RSA keys, adding a new listening thread, and registering the client... and consequently that was going quite badly (it was noticeable clearly when I hit the button it took a while). Also, if you generate 1000 payloads... having 1000 threads there, it's not going to go very well.

With this PR, there is now only 1 client and 1 polling thread, and all payloads share the same correlationID prefix. If you want to unregister the client and register a new one, changing the correlationID, just press "Update settings" in the configuration tab

Add poll now button

The title is self-explanatory

Add poll interval config

The title is self-explanatory

Allow column/row selection for copy values.

Before you could only select rows, now you can select rows, columns and specific cells. Allowing you to copy values from the log table.

Add workflow to generate release JAR

Add a GitHub action to generate the release file automatically when something is pushed to master.

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.

1 participant