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

App freezes when trying to reconnect after host key changes #1948

Open
MatzeKitt opened this issue Jan 8, 2024 · 1 comment
Open

App freezes when trying to reconnect after host key changes #1948

MatzeKitt opened this issue Jan 8, 2024 · 1 comment
Labels
Bug Something isn't working PR Welcome Issues and fixes available for wide community to help us move forward by creating a PR with solution

Comments

@MatzeKitt
Copy link

MatzeKitt commented Jan 8, 2024

  • Sequel Ace Version (including build number): 4.0.13 Build 20062
  • Sequel Ace Source (App Store/GitHub/Homebrew): GitHub
  • macOS Version: 14.2.1 (23C71)
  • Processor Type (Intel/Apple): Apple
  • macOS Localization: German

Is Issue Present in Latest Beta?

Yes

Description
If a host key changes (in our case as part of load balancing) and you try to interact with the data within an open connection, Sequel Ace does correctly recognize this, but a reconnect is impossible since, as soon as the popup notifying about this change is displayed, the app freezes with nearly 100 % CPU core utilization and you cannot click on any of the buttons in the popup.

Bildschirmfoto 2024-01-08 um 09 23 07

Bildschirmfoto 2024-01-08 um 09 23 49

Steps To Reproduce

  1. Open a connection
  2. Change the host key of the connected machine
  3. Refresh the table view

Expected Behaviour

The buttons popup can be clicked and a reconnect works without an app freeze.

Related Issues

Maybe related: #1673

Additional Context

@Jason-Morcos Jason-Morcos added Bug Something isn't working PR Welcome Issues and fixes available for wide community to help us move forward by creating a PR with solution labels Jan 9, 2024
@Jason-Morcos
Copy link
Member

This feels like a pretty niche one, but would be happy to review and merge a fixing PR here if someone wants to make one :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working PR Welcome Issues and fixes available for wide community to help us move forward by creating a PR with solution
Projects
None yet
Development

No branches or pull requests

2 participants