Skip to content

Update pre-releases api url#1667

Closed
yosifkit wants to merge 1 commit into
docker-library:masterfrom
infosiftr:url-change
Closed

Update pre-releases api url#1667
yosifkit wants to merge 1 commit into
docker-library:masterfrom
infosiftr:url-change

Conversation

@yosifkit

@yosifkit yosifkit commented Jun 23, 2026

Copy link
Copy Markdown
Member

php.net does have a redirect from release-candidates to pre-release-builds, but it stopped passing along query parameters on the 19th of June (but I don't see a code change that causes it). So, we'll just use the newer route.

Without this, the update.sh job fails with the following:

jq: parse error: Invalid numeric literal at line 1, column 10
curl: (23) Failure writing output to destination, passed 8192 returned 0

testing

$ curl -fsSL https://www.php.net/release-candidates.php?format=json | head -n2
<!DOCTYPE html>
<html xmlns="http://www.w3.org/1999/xhtml" lang="en">
$ curl -fsSL https://www.php.net/pre-release-builds.php?format=json | jq | head -n2
{
  "8.2.27": {

@jnoordsij jnoordsij left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, totally missed this and just opened #1668 which basically does this as well. You may also want to include the other URL reference in the comment as good practice (see jnoordsij@b4e4077).

Feel free to merge this instead and close #1668 to let the bot do its thing and update afterwards.

@TimWolla

Copy link
Copy Markdown
Contributor

cc @derickr

@derickr

derickr commented Jun 24, 2026

Copy link
Copy Markdown

Sorry about that!

Funnily, that redirect in that file got never triggered, as the array key needed to match the full URL (including query string).

We did change CDN (and origin server), but AFAIK we didn't make any changes wrt this.

I have now fixed this with Apache style redirects, so it should now work again.

Of course, switching to the right/new URI does save you an HTTP redirect.

@yosifkit yosifkit mentioned this pull request Jun 24, 2026
@yosifkit

Copy link
Copy Markdown
Member Author

Thanks everyone! ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants