8000 244332 - This is for a bug I rased about Rdp and multiple sessions by somanysteves · Pull Request #247528 · microsoft/vscode · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

244332 - This is for a bug I rased about Rdp and multiple sessions #247528

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

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

somanysteves
Copy link
@somanysteves somanysteves commented Apr 27, 2025

If you

  1. log into two sessions with the same user
  2. launch vs code from the command line in session 1
  3. launch vs code from the command line in session 2

The second instance starts in the first session.

I assume this can't be the intended behavior as it's weird to launch an app in a place you won't see it.

This fix scopes the named pipe to include the name of the session, which is unique to that specific session, regardless of user.

@somanysteves
Copy link
Author

This is for the issue I raised a bit ago: #244332

somanysteves and others added 2 commits April 27, 2025 19:56
If you
1) log into two sessions with the same user
2) launch vs code from the command line in session 1
3) launch vs code from the command line in session 2
The second instance starts in the first session.
I assume this can't ben the intended behavior as it's weird to launch an
app in a place you won't see it.

This fix scopes the named pipe to include the name of the session, which
is unique to that specific session, regardless of user.
@somanysteves
Copy link
Author

@joyceerhl is there any other action required on my end for this?

@joyceerhl joyceerhl assigned connor4312 and unassigned joyceerhl Apr 29, 2025
@connor4312 connor4312 assigned bpasero and unassigned connor4312 Apr 29, 2025
@bpasero
Copy link
Member
bpasero commented Apr 30, 2025

I think this is not the right fix: the socket needs to stay symmetric with all other folders for the instance (specifically user data path), otherwise we risk that 2 separate instances open over the same user data dir, that is actually what we try to prevent here.

@somanysteves
Copy link
Author

@bpasero thanks for the feedback. I was worried it felt too easy.
I am new to this code base though...
Any suggestions on how I would go about implementing this correctly?
The current behavior is very frustrating for my work flow. If I need vs in a different session, I need to kill all existing vscode processes before I can launch it.

@bpasero
Copy link
Member
bpasero commented Apr 30, 2025

I believe this cannot be easily fixed automatically because it might make the wrong assumptions for users that only use 1 session.

Have you considered a portable setup? See https://code.visualstudio.com/docs/editor/portable

@somanysteves
Copy link
Author

@bpasero - Thanks for the suggestion. I downloaded vscode portable and tried to repro, however it still does the same "wrong session launch" behavior.

I think I may be confused though. I don't follow how this could break single session functionality. In the case of the single session, the session name will be consistent.

I have tested that scenario and it continues to work as expected.

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.

4 participants
0