-
-
Notifications
You must be signed in to change notification settings - Fork 10.8k
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
todesk: update livecheck #172925
todesk: update livecheck #172925
Conversation
The existing `livecheck` block for `todesk` checks an upstream release notes page but the response body only contains a `<script>` element when checked with curl, so it gives an `Unable to get versions` error. This seems to be true regardless of headers (e.g., user agent, etc.). This updates the `livecheck` block to check the download page and match the version from the pkg URL (found in some JavaScript code).
Looks like the SHA256 changed since this was last updated:
|
|
8c4e69c
to
91e2312
Compare
@razvanazamfirei we shouldn't use |
Sorry, accidentally deleted the branch! |
Seems that Todesk is blocking users from downloading with Homebrew.
The obfuscated JS code keeps changing so the SHA256 also keeps changing. Also, I tested a few times. The only way to download Todesk from command line is using a proper browser UA and a Chinese internet environment.
Missing any of these, the url will only return garbage. |
Thanks @Aaron-212 for looking into this. Unfortunately, that may require removing this software the repository if we can't get a stable download. |
Great, now it works 👍 |
Important: Do not tick a checkbox if you haven’t performed its action. Honesty is indispensable for a smooth review process.
In the following questions
<cask>
is the token of the cask you're submitting.After making any changes to a cask, existing or new, verify:
brew audit --cask --online <cask>
is error-free.brew style --fix <cask>
reports no offenses.Additionally, if adding a new cask:
brew audit --cask --new <cask>
worked successfully.HOMEBREW_NO_INSTALL_FROM_API=1 brew install --cask <cask>
worked successfully.brew uninstall --cask <cask>
worked successfully.The existing
livecheck
block fortodesk
checks an upstream release notes page but the response body only contains a<script>
element when checked with curl, so it gives anUnable to get versions
error. This seems to be true regardless of headers (e.g., user agent, etc.).This updates the
livecheck
block to check the download page and match the version from the pkg URL (found in some JavaScript code).