Skip to content

[Bug]: Update popup notification although --disable-update-check #4826

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

Closed
3 tasks done
reaseno opened this issue Feb 4, 2022 · 7 comments · Fixed by #4997
Closed
3 tasks done

[Bug]: Update popup notification although --disable-update-check #4826

reaseno opened this issue Feb 4, 2022 · 7 comments · Fixed by #4997
Assignees
Labels
bug Something isn't working
Milestone

Comments

@reaseno
Copy link

reaseno commented Feb 4, 2022

Is there an existing issue for this?

  • I have searched the existing issues

OS/Web Information

  • Web Browser: Vivaldi
  • Local OS: Win10
  • Remote OS: Debian
  • Remote Architecture:
  • code-server --version: 4.0.1

Steps to Reproduce

  1. run with: code-server --disable-update-check

Expected

No update check and notification.

Actual

Update check and a popup notification for code-server 4.0.2 within the IDE at the bottom right.

Logs

No response

Screenshot/Video

No response

Does this issue happen in VS Code?

  • I cannot reproduce this in VS Code.

Are you accessing code-server over HTTPS?

  • I am using HTTPS.

Notes

No response

@reaseno reaseno added bug Something isn't working triage This issue needs to be triaged by a maintainer labels Feb 4, 2022
@jsjoeio
Copy link
Contributor

jsjoeio commented Feb 4, 2022

This is not something we support unfortunately (running code-server --help shows you the flags we support).

You should be able to disable updates in your VS Code Settings by setting it to "none"

image

@jsjoeio jsjoeio closed this as completed Feb 4, 2022
@jsjoeio jsjoeio removed bug Something isn't working triage This issue needs to be triaged by a maintainer labels Feb 4, 2022
@reaseno
Copy link
Author

reaseno commented Feb 5, 2022

I get this parameter with "code-server --help":

--disable-update-check Disable update check. Without this flag, code-server checks every 6 hours against the latest github release and then notifies you once every week that a new release is available.

And update-mode was set to none.

@jsjoeio jsjoeio reopened this Feb 7, 2022
@jsjoeio
Copy link
Contributor

jsjoeio commented Feb 7, 2022

My apologies...I didn't realize we still had that.

This flag is before my time. Did it used to work? Do you know what it should do?

I took a look through the codebase and I can't find any logic related to it 🤔 My thinking would be that it would set update-mode to none, but maybe it's supposed to do something else.

@jsjoeio jsjoeio added bug Something isn't working needs-investigation This issue needs to be further investigated labels Feb 7, 2022
@jsjoeio jsjoeio added this to the On Deck milestone Feb 7, 2022
@reaseno
Copy link
Author

reaseno commented Feb 7, 2022

All I know is that I get an update notification even though both options are set. So one of them doesn't seem to work anymore 🤷‍♂️

@jsjoeio
Copy link
Contributor

jsjoeio commented Feb 7, 2022

Well that surely is annoying, isn't it? Sorry about that!

Maybe we lost some of that logic in 4.0.

@code-asher any ideas on how this worked previously?

@code-asher
Copy link
Member

It would appear this got left out in the rebase.

@code-asher
Copy link
Member

We will need to take a similar route to ignore-last-opened.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants