Skip to content

Clear session on disconnect - #1941

Merged
tadaskay merged 1 commit into
masterfrom
tk/fixes
Mar 28, 2020
Merged

Clear session on disconnect#1941
tadaskay merged 1 commit into
masterfrom
tk/fixes

Conversation

@tadaskay

Copy link
Copy Markdown
Contributor

No description provided.

This caused old session being left intact when disconnected.
When client attempts to connect to another provider, he is shown an old session/proposal (via SSE).
@tadaskay tadaskay changed the title Clear proposal state (SSE) when disconnected Clear session on disconnect Mar 28, 2020
@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #1941 into master will increase coverage by 0.49%.
The diff coverage is 51.12%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1941      +/-   ##
==========================================
+ Coverage   47.48%   47.98%   +0.49%     
==========================================
  Files         286      287       +1     
  Lines       12108    12617     +509     
==========================================
+ Hits         5750     6054     +304     
- Misses       5878     5999     +121     
- Partials      480      564      +84     
Impacted Files Coverage Δ
core/connection/stats.go 70.00% <ø> (ø)
core/service/subscription.go 0.00% <0.00%> (ø)
mobile/mysterium/entrypoint.go 0.00% <0.00%> (ø)
nat/traversal/noop.go 0.00% <0.00%> (ø)
services/wireguard/connection/connection.go 58.95% <0.00%> (-2.12%) ⬇️
session/dto.go 42.85% <0.00%> (-17.15%) ⬇️
session/pingpong/exchange_messaging.go 0.00% <0.00%> (ø)
session/pingpong/factory.go 0.00% <0.00%> (ø)
session/pingpong/invoice_messaging.go 0.00% <0.00%> (ø)
tequilapi/client/dto.go 0.00% <ø> (ø)
... and 31 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6e89c61...5993eaa. Read the comment docs.

@tadaskay
tadaskay merged commit fc0fc64 into master Mar 28, 2020
@tadaskay
tadaskay deleted the tk/fixes branch March 28, 2020 18:56
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