Skip to content
This repository has been archived by the owner on Sep 29, 2024. It is now read-only.

Some servers might send mixed DATA_V1 / DATA_V2 packets #30

Closed
keeshux opened this issue Oct 17, 2018 · 1 comment
Closed

Some servers might send mixed DATA_V1 / DATA_V2 packets #30

keeshux opened this issue Oct 17, 2018 · 1 comment
Assignees
@keeshux
Labels
bug Something isn't working
Milestone
1.2.0

Comments

@keeshux
Copy link
Member

keeshux commented Oct 17, 2018

Mullvad pushes a peer-id then sends DATA_V1 packets (?). The current code can't handle a mix of the two types. The effect is an abrupt disconnection due to a decryption error (103), because the packet has an unexpected format.
@keeshux keeshux added the bug Something isn't working label Oct 17, 2018
@keeshux keeshux mentioned this issue Oct 18, 2018
Closed
keeshux added a commit that referenced this issue Oct 19, 2018
@keeshux
Handle encryption/peer-id in a stateless manner
bff9352 
Fixes #30
@keeshux keeshux mentioned this issue Oct 19, 2018
Merged
@keeshux
Copy link
Member Author

keeshux commented Oct 19, 2018

#31 might be related?

@keeshux keeshux self-assigned this Oct 23, 2018
@keeshux keeshux added this to the 1.2.0 milestone Oct 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@keeshux keeshux

Labels
bug Something isn't working
Projects
None yet
Milestone
1.2.0
Development

No branches or pull requests
1 participant
@keeshux