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

Reset port when chaning transport protocol #5445

Merged

Conversation

raksooo
Copy link
Member

@raksooo raksooo commented Nov 16, 2023

Issue introduced in #5376. When changing the OpenVPN transport protocol the port isn't reset resulting in the blocked state if connected. This is fixed by resetting the port when changing transport protocol.


This change is Reviewable

Copy link

linear bot commented Nov 16, 2023

DES-458 Port isn't reset when changing transport protocol

When using OpenVPN and selecting a transport protocol and a port, and then chagning to the other transport protocol (not automatic). After doing this the app ends up in the blocked state due to the port being kept. The port needs to be reset when switching protocol.

OpenVPN constraints
    Port:                   1196
    Transport:              TCP

@raksooo raksooo requested a review from dlon November 16, 2023 07:08
Copy link
Member

@dlon dlon left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@raksooo raksooo force-pushed the port-isnt-reset-when-changing-transport-protocol-des-458 branch from 3ada8a3 to 802e500 Compare November 16, 2023 07:54
@raksooo raksooo merged commit 67710f3 into main Nov 16, 2023
4 checks passed
@raksooo raksooo deleted the port-isnt-reset-when-changing-transport-protocol-des-458 branch November 16, 2023 07:59
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.

2 participants