Skip to content
This repository was archived by the owner on Oct 22, 2024. It is now read-only.

fix: resolve pairing issues in firefox#291

Merged
matthewwalsh0 merged 4 commits intomain-desktopfrom
fix/278-firefox-pairing
Dec 6, 2022
Merged

fix: resolve pairing issues in firefox#291
matthewwalsh0 merged 4 commits intomain-desktopfrom
fix/278-firefox-pairing

Conversation

@matthewwalsh0
Copy link
Copy Markdown
Member

@matthewwalsh0 matthewwalsh0 commented Dec 1, 2022

Overview

Resolve issues preventing successful pairing in Firefox.

Changes

App

  • Add error handling after failed web socket stream initialisation.

Common

  • Ignore acknowledgement messages in desktop connection state stream.
  • Ignore all state and disable messages until desktop connection is paired.
  • Remove connection during initialisation and always lazily connect when needed.
  • Add timeout to encryption handshake.
  • Add automatic resend to both sides of encryption handshake.
  • Add timeout and error handling to web socket stream writes.
  • Add error handling after failed web socket stream initialisation.

@matthewwalsh0 matthewwalsh0 changed the base branch from main-desktop to chore/225-common-migration-3 December 1, 2022 12:19
@matthewwalsh0 matthewwalsh0 changed the title Fix pairing in FireFox fix: resolve pairing issues in firefox Dec 1, 2022
@matthewwalsh0 matthewwalsh0 force-pushed the fix/278-firefox-pairing branch from 3c594c5 to 7b8ec98 Compare December 1, 2022 17:31
@matthewwalsh0 matthewwalsh0 marked this pull request as ready for review December 2, 2022 08:50
@matthewwalsh0 matthewwalsh0 requested a review from a team December 2, 2022 08:50
Copy link
Copy Markdown
Contributor

@bergarces bergarces left a comment

Choose a reason for hiding this comment

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

I can't find anything to comment on. It feels like a great improvement and we will have better error reporting if/when something goes wrong initialising a websocket connection.

@matthewwalsh0 matthewwalsh0 force-pushed the chore/225-common-migration-3 branch from 6a436c0 to de384c3 Compare December 2, 2022 22:03
@matthewwalsh0 matthewwalsh0 force-pushed the fix/278-firefox-pairing branch from f1029b0 to a56f111 Compare December 2, 2022 22:27
Base automatically changed from chore/225-common-migration-3 to main-desktop December 6, 2022 13:30
Remove desktop connection on extension startup.
Add encryption handshake timeout and graceful web socket close.
Add automatic retry to both sides of encryption handshake.
@matthewwalsh0 matthewwalsh0 force-pushed the fix/278-firefox-pairing branch from a56f111 to 9f7d718 Compare December 6, 2022 13:46
@matthewwalsh0 matthewwalsh0 merged commit bcb6042 into main-desktop Dec 6, 2022
@matthewwalsh0 matthewwalsh0 deleted the fix/278-firefox-pairing branch December 6, 2022 15:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants