Skip to content

Conversation

@SVillette
Copy link

Q A
Branch? main
Tickets -
License MIT
Doc PR api-platform/docs#...

This PR replaces the usage of parse_url() method by the new rfc 3986 URI API on PHP8.5.
cc @soyuka this is the subject we talk about at the SymfonyCon hackday.

Still in draft as it's missing some tests.

@SVillette SVillette marked this pull request as draft November 29, 2025 13:59
@SVillette SVillette force-pushed the feature-use-rfc3986-uri branch 2 times, most recently from c91971d to c442327 Compare November 30, 2025 10:38
@soyuka soyuka marked this pull request as ready for review December 11, 2025 14:05
@soyuka
Copy link
Member

soyuka commented Dec 11, 2025

could you maybe rebase this?

@SVillette SVillette force-pushed the feature-use-rfc3986-uri branch from c442327 to bc53a3b Compare December 12, 2025 10:01
@SVillette
Copy link
Author

@soyuka PR is rebased.
However these changes break a test in ApiPlatform\Metadata\Tests\Util\IriHelperTest as http:/// is considered valid with the new Uri\Rfc3986\Uri class. WDYT about it?

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.

2 participants