Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

fix: remote ide no longer reconnects after plugin upgrade#167

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
fioan89 merged 1 commit intomainfromfix-ide-connection-lost-after-upgrade
Aug 11, 2025

Conversation

fioan89
Copy link
Collaborator

@fioan89fioan89 commentedAug 7, 2025
edited
Loading

When the plugin is upgraded while JBClient is connected to a remote dev server via the Coder SSH proxy/tunnel, the upgrade process kills and re-establishes the SSH connection. However, JBClient/Toolbox fails to detect the restored connection and reports "Toolbox: Target environment com.coder.toolbox:bobiverse-bob.dev not found" error.

While digging into the Toolbox bytecode—specificallyClientOverSshTunnelConnector — I realized the issue likely stems from an incorrect equals implementation in our custom SSH connection info object. In short, when a plugin upgrade terminates the SSH tunnel, the connector’s monitoring logic correctly detects the lost connection and waits. But when the SSH connection is re-established, the monitoring logic fails to recognize it as a valid replacement, because equals is still using the defaultObject#equals rather than a proper value-based implementation.

Unfortunately, I wasn’t able to properly test this—specifically, upgrading from a version without the fix to one that includes it—because all Toolbox marketplace feeds are signed, preventing us from using a tool like mitmproxy to serve a locally modified plugin version. Given that, I propose releasing the change first and then performing the upgrade test to confirm the fix.

When the plugin is upgraded while JBClient is connected to a remote dev server via the Coder SSH proxy/tunnel,the upgrade process kills and re-establishes the SSH connection. However, JBClient/Toolbox fails to detect the restoredconnection and reports "Toolbox: Target environment com.coder.toolbox:bobiverse-bob.dev not found" error.While digging into the Toolbox bytecode—specifically ClientOverSshTunnelConnector—I realized the issue likely stemsfrom an incorrect equals implementation in our custom SSH connection info object. In short, when a plugin upgrade terminatesthe SSH tunnel, the connector’s monitoring logic correctly detects the lost connection and waits. But when the SSH connectionis re-established, the monitoring logic fails to recognize it as a valid replacement, because equals is still using thedefault Object#equals rather than a proper value-based implementation.Unfortunately, I wasn’t able to properly test this—specifically, upgrading from a version without the fix to one that includesit—because all Toolbox marketplace feeds are signed, preventing us from using a tool like mitmproxy to serve a locally modifiedplugin version. Given that, I propose releasing the change first and then performing the upgrade test to confirm the fix.
@fioan89fioan89 marked this pull request as ready for reviewAugust 7, 2025 23:04
@fioan89
Copy link
CollaboratorAuthor

I also requested a review/opinion from JetBrains on this matter. Ideally this should be tested before merging.

Copy link
Member

@code-ashercode-asher left a comment

Choose a reason for hiding this comment

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

Interesting, that makes a lot of sense!

@fioan89fioan89 merged commit6d509d6 intomainAug 11, 2025
6 checks passed
@fioan89fioan89 deleted the fix-ide-connection-lost-after-upgrade branchAugust 11, 2025 17:31
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@code-ashercode-ashercode-asher approved these changes

@matifalimatifaliAwaiting requested review from matifali

@f0sself0sselAwaiting requested review from f0ssel

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@fioan89@code-asher

[8]ページ先頭

©2009-2025 Movatter.jp