-
-
Notifications
You must be signed in to change notification settings - Fork 279
Update bedrock-entry.sh, fix data-platform missing error
#525
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
Conversation
itzg
left a comment
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.
Thanks for fixing the root cause. Quite annoying they changed after 5+ years.
|
Tried this locally, and it's not fixing it; the restify command still isn't returning the right URL. I wonder if it is because they wrapped |
|
I don't see any extra double quotes, but I definitely need to investigate if there's a bug in restify |
|
Is this new? Looks like it's now referring to an API to get the link URLs: Which makes this easy: |
|
Awesome find. Yeah, that's much easier to work with. Otherwise it looked like their page has broken scraping the link entirely: |
|
If you want to change to use the curl+jq approach, then that would be awesome. |
|
...I can later remove the use of https://github.com/itzg/mc-bds-helper since that was just to cache the scraping result as a fallback. |
|
Cool! I'll put a PR in... will add some validation in case the platform is not present too EDIT - will fix this PR |
|
This works, but we need to figure out how to lookup custom version URLs via the new API to re-add support for versions other than latest. Also, if curl or jq fails, then the entry script fails. I'll fix it to pass an error message through gracefully so it can use a cached version. But for anyone stuck with a broken server, this will get them going! |
itzg
left a comment
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.
Cool! Yeah I would expect the latest version lookup to be the 99% use case, so this is fantastic at this point.
See #524 for issue.