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
Add DownloadPageAPI to the autoupdater to download from HTML web pages #2041
base: master
Are you sure you want to change the base?
Conversation
8b9b8a4
to
1db1634
Compare
I'm not saying it's a bad idea, I'm saying it's brittle as is web scraping in general 🤔 |
The version_regex is matching against the link "content" (name ?). |
Ah yes, agreed! But it's kind of an internal tool, if it breaks, no user will complain, so i'd say it's okay to have a not-really-hardened tool there. |
1db1634
to
1f65ac2
Compare
1f65ac2
to
963fa16
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM this handy shit
FWIW, I would probably need that for elasticsearch8. |
Tested with Jitsi and this configuration: