Skip to content

Conversation

@rgroh1996
Copy link
Contributor

The PPK2 was not reset when the object was deleted. The PPK2 is reset with the PPK2 RESET command. This allows a new connection to be established immediately.

@wlgrd
Copy link
Collaborator

wlgrd commented Nov 21, 2023

Nice addition. Please update your commit title to adhere to github's max length.
Suggestion, do git commit --amend, and change to

send reset command in destructor

The PPK2 was not reset when the object was deleted.
The PPK2 is reset with the PPK2 RESET command.
This allows a new connection to be established immediately.

then do a git push --force after saving the new commit

The PPK2 was not reset when the object was deleted.
The PPK2 is reset with the PPK2 RESET command.
This allows a new connection to be established immediately.
@rgroh1996 rgroh1996 force-pushed the enhancement/destructor-ppk2-reset branch from f81d579 to abefe62 Compare November 22, 2023 15:35
@wlgrd wlgrd self-requested a review November 30, 2023 10:10
@wlgrd wlgrd merged commit e3fb545 into IRNAS:master Nov 30, 2023
@juhhov
Copy link

juhhov commented Aug 26, 2024

Could this be included into release? @wlgrd @NejcKle

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