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

🐛 Bug Report: Doctor Always Fails to Check for Updates #7717

Open
2 tasks done
ItzNotABug opened this issue Mar 3, 2024 · 1 comment
Open
2 tasks done

🐛 Bug Report: Doctor Always Fails to Check for Updates #7717

ItzNotABug opened this issue Mar 3, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@ItzNotABug
Copy link
Contributor

ItzNotABug commented Mar 3, 2024

👟 Reproduction steps

  1. Run docker exec appwrite doctor
  2. In the end, the command always fails to check for the latest app version

Its possible this is the cause here -
https://github.com/appwrite/appwrite/blob/9e4250ad9fcb6b383c7edfd398e263de8fea8a5b/src/Appwrite/Platform/Tasks/Doctor.php#L251C9-L268C10

I can see the variables being set via docker exec appwrite env -

_APP_VERSION=1.4.12
_APP_HOME=https://appwrite.io

The implementation -

$version = \json_decode(@\file_get_contents(App::getEnv('_APP_HOME', 'http:https://localhost') . '/version'), true);

seems to be getting contents from this url or file (based on the parameters) - https://appwrite.io/version.
If this is a network fetch, this results in a 404.

👍 Expected behavior

Provide info. if there's an update available or not. Maybe use the Github Releases to look for the latest release tag.

👎 Actual Behavior

Throws an error via console - Failed to check for a newer version.

🎲 Appwrite version

Version 1.4.x

💻 Operating system

Linux

🧱 Your Environment

Localhost & Ubuntu 22.04.3 LTS.

👀 Have you spent some time to check if this issue has been raised before?

  • I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

@ItzNotABug ItzNotABug added the bug Something isn't working label Mar 3, 2024
@stnguyen90
Copy link
Contributor

@ItzNotABug, thanks for raising this issue! 🙏🏼 We used to have version checks in the Appwrite Console and the Doctor command before we upgraded the Console. We still have plans to bring that back and will probably fix the problem in the Doctor when we do that.

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

No branches or pull requests

2 participants