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

"Retry as sudo" claims it "failed to save" but it actually succeeded #93602

Closed
Nowaker opened this issue Mar 27, 2020 · 2 comments
Closed

"Retry as sudo" claims it "failed to save" but it actually succeeded #93602

Nowaker opened this issue Mar 27, 2020 · 2 comments
Assignees
Labels
author-verification-requested Issues potentially verifiable by issue author bug Issue identified by VS Code Team member as probable bug verified Verification succeeded workbench-run-as-admin Issues concerning running as administrator
Milestone

Comments

@Nowaker
Copy link

Nowaker commented Mar 27, 2020

  • VSCode Version:
Version: 1.43.1
Commit: fe22a9645b44368865c0ba92e2fb881ff1afce94
Date: 2020-03-18T07:17:38.324Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Linux x64 5.5.11-arch1-1
  • OS Version: Arch Linux 20200317

Steps to Reproduce:

  1. Run VS Code as non-root.
  2. Edit a file owned by root.
  3. Try to save.
  4. Error. Click "Retry as sudo".
    image
  5. Authenticate successfully.
  6. See the error. image
  7. Inspect the file with an external editor. File has changed. It had succeeded contrary to what VS Code said.

Conclusion:

The printed "errors" were just warnings. VS Code thought what's in stderr is errors. Which is wrong as stderr is just warnings unless exit status is non-zero.

My environment is very unusual: using Cinnamon (a GTK+ desktop) with kde-sudo. While I don't know why kde-sudo was used instead of its GTK+ equivalent, that's what VS Code used and I had no choosing over it.

Does this issue occur when all extensions are disabled?: Yes

@Nowaker Nowaker changed the title "Retru as sudo" claims it "failed to save" but it actually succeeded "Retry as sudo" claims it "failed to save" but it actually succeeded Mar 27, 2020
@bpasero bpasero added this to the March 2020 milestone Mar 29, 2020
@bpasero bpasero added workbench-run-as-admin Issues concerning running as administrator bug Issue identified by VS Code Team member as probable bug labels Mar 29, 2020
@bpasero bpasero added the author-verification-requested Issues potentially verifiable by issue author label Mar 30, 2020
@Nowaker
Copy link
Author

Nowaker commented Mar 30, 2020

Thanks @bpasero!

@github-actions
Copy link

This bug has been fixed in to the latest release of VS Code Insiders!

@Nowaker, you can help us out by confirming things are working as expected in the latest Insiders release. If things look good, please leave a comment with the text /verified to let us know. If not, please ensure you're on version d7d1147 of Insiders (today's or later - you can use Help: About in the command pallete to check), and leave a comment letting us know what isn't working as expected.

Happy Coding!

@connor4312 connor4312 added the verified Verification succeeded label Apr 3, 2020
@github-actions github-actions bot locked and limited conversation to collaborators May 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
author-verification-requested Issues potentially verifiable by issue author bug Issue identified by VS Code Team member as probable bug verified Verification succeeded workbench-run-as-admin Issues concerning running as administrator
Projects
None yet
Development

No branches or pull requests

3 participants