Skip to content

Conversation

adietish
Copy link
Collaborator

@adietish adietish commented Aug 8, 2025

When trying to connect to a workspace via link and it then fails, you get a BindException if you try again. That's because the server socket, that's used to forward, isn't closed if connecting fails.

@codecov-commenter
Copy link

codecov-commenter commented Aug 8, 2025

Codecov Report

❌ Patch coverage is 0% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (71098f6) to head (40c209d).
⚠️ Report is 194 commits behind head on main.

Files with missing lines Patch % Lines
...com/redhat/devtools/gateway/DevSpacesConnection.kt 0.00% 8 Missing ⚠️
...tlin/com/redhat/devtools/gateway/openshift/Pods.kt 0.00% 3 Missing ⚠️
...t/devtools/gateway/server/RemoteIDEServerStatus.kt 0.00% 1 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##            main    #150    +/-   ##
======================================
  Coverage   0.00%   0.00%            
======================================
  Files          4      25    +21     
  Lines         26     887   +861     
  Branches       0     118   +118     
======================================
- Misses        26     887   +861     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@adietish adietish marked this pull request as ready for review August 8, 2025 15:58
@adietish adietish requested a review from azatsarynnyy as a code owner August 8, 2025 15:58
@adietish adietish changed the title fix: avoid bind exception when retrying after failure fix: avoid bind exception when retrying to connect after failure Aug 8, 2025
@adietish adietish changed the title fix: avoid bind exception when retrying to connect after failure fix: avoid bind exception when reconnecting after failure Aug 8, 2025
@adietish adietish force-pushed the avoid_bindexception branch 3 times, most recently from 9e8e4b3 to ef92153 Compare August 8, 2025 17:59
@adietish adietish marked this pull request as draft August 11, 2025 21:44
@adietish adietish force-pushed the avoid_bindexception branch 2 times, most recently from 8c0f8b4 to a2c90ed Compare August 15, 2025 19:19
@adietish adietish force-pushed the avoid_bindexception branch from a2c90ed to 40c209d Compare August 18, 2025 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants