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

When the command longer than 1023 runInTerminal fail #134324

Closed
chianquan opened this issue Oct 2, 2021 · 5 comments
Closed

When the command longer than 1023 runInTerminal fail #134324

chianquan opened this issue Oct 2, 2021 · 5 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug *duplicate Issue identified as a duplicate of another issue(s) terminal Integrated terminal issues

Comments

@chianquan
Copy link

chianquan commented Oct 2, 2021

Does this issue occur when all extensions are disabled?: Yes/No
The bug is found when use extensions,so I can't disable all extensions.

  • VS Code Version:1.60.2 (Universal)
  • OS Version: Darwin x64 20.2.0

Steps to Reproduce:

  1. use vscode-js-debug plugin
  2. make env PATH logger enough to cause target command longer than 1023 .
  3. run js debug. It will halt when the debug plugin first send runInTerminal request.the terminal leave over an command length 1023(It seems cut by something.)

image

@roblourens
Copy link
Member

This sounds familiar... do you know what the issue is here @connor4312?

@connor4312
Copy link
Member

Sounds familiar to, but I don't see any open issue for it and can readily reproduce it.

Issue in core so assigning to the both of us 🙂

@connor4312 connor4312 added the debug Debug viewlet, configurations, breakpoints, adapter issues label Oct 11, 2021
@roblourens
Copy link
Member

I'm pretty sure we are sending the right text to the terminal via sendText, but it's a little weird, it goes away when I am debugging. Do you know @Tyriar?

Once I got text missing from the middle instead of just a truncated command /Users/roblou/.nvm/versions/node/v16.4.0/bin/node: invalid value for NODE_OPTIONS (unterminated string)

@roblourens roblourens added the bug Issue identified by VS Code Team member as probable bug label Oct 11, 2021
@connor4312
Copy link
Member

I can verify the right text is being sent here:

terminal.sendText(command);

I can also repro by making an extension that simply does:

const term = vscode.window.createTerminal();
term.show();
term.sendText(`echo "${'hello world'.repeat(500)}" | wc -c`)

image

So this seems like a terminal issue

@connor4312 connor4312 assigned Tyriar and meganrogge and unassigned roblourens and connor4312 Oct 14, 2021
@connor4312 connor4312 added terminal Integrated terminal issues and removed debug Debug viewlet, configurations, breakpoints, adapter issues labels Oct 14, 2021
@Tyriar
Copy link
Member

Tyriar commented Oct 19, 2021

/duplicate #127699

@Tyriar Tyriar closed this as completed Oct 19, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Dec 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug *duplicate Issue identified as a duplicate of another issue(s) terminal Integrated terminal issues
Projects
None yet
Development

No branches or pull requests

6 participants
@roblourens @Tyriar @connor4312 @chianquan @meganrogge and others