Skip to content

fix: remove unnecessary configuration due new mercure_publish() PHP function#869

Draft
7-zete-7 wants to merge 1 commit intodunglas:mainfrom
7-zete-7:builtin-mercure-func
Draft

fix: remove unnecessary configuration due new mercure_publish() PHP function#869
7-zete-7 wants to merge 1 commit intodunglas:mainfrom
7-zete-7:builtin-mercure-func

Conversation

@7-zete-7
Copy link
Contributor

@7-zete-7 7-zete-7 commented Nov 24, 2025

With the addition of the new PHP function mercure_publish() in php/frankenphp#1927, the use of the closure via the php:80 host is no longer necessary. The php:80 can be removed and the MERCURE_URL env can be removed empty.

@7-zete-7
Copy link
Contributor Author

The actual symfony/mercure-bundle recipe still requires the MERCURE_URL env. I'm guessing it's not yet possible to prevent using the environment variable in this recipe, as other environments expect this behavior.

@7-zete-7
Copy link
Contributor Author

Waiting until symfony/recipes#1514 is merged.

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.

1 participant