Control: retitle: -1 unblock: php-embed/4.4.17-1 Hi,
On 13/05/2025 18:32, David Prévot wrote: […]
[ Impact ] Not much impact expected.
Obviously, I was wrong, and upstream just released a s/+/-/ fix on their recent change.
[ Tests ] Tests at build time, as well as autopkgtest (and thus reverse-dependencies have also been tested). [ Risks ] Code change is trivial. [ Checklist ] [x] all changes are documented in the d/changelog [x] I reviewed all changes and I approve them [x] attach debdiff against the package in testing [ Other info ] I’d have it uploaded to experimental if I had realized sooner I needed to bug you, sorry.
Sorry, sorry, sorry… unblock php-embed/4.4.17-1
diff -Nru php-embed-4.4.15/CHANGELOG.md php-embed-4.4.17/CHANGELOG.md --- php-embed-4.4.15/CHANGELOG.md 2025-01-02 17:53:09.000000000 +0100 +++ php-embed-4.4.17/CHANGELOG.md 2025-05-13 14:42:29.000000000 +0200 @@ -4,6 +4,14 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/) and this project adheres to [Semantic Versioning](http://semver.org/). +## [4.4.17] - 2025-05-13 +### Fixed +- Adapters hostname detection [#556]. + +## [4.4.16] - 2025-05-09 +### Fixed +- Adapters hostname detection [#555]. + ## [4.4.15] - 2025-01-02 ### Fixed - Type bug [#553]. @@ -254,7 +262,11 @@ [#548]: https://github.com/oscarotero/Embed/issues/548 [#551]: https://github.com/oscarotero/Embed/issues/551 [#553]: https://github.com/oscarotero/Embed/issues/553 +[#555]: https://github.com/oscarotero/Embed/issues/555 +[#556]: https://github.com/oscarotero/Embed/issues/556 +[4.4.17]: https://github.com/oscarotero/Embed/compare/v4.4.16...v4.4.17 +[4.4.16]: https://github.com/oscarotero/Embed/compare/v4.4.15...v4.4.16 [4.4.15]: https://github.com/oscarotero/Embed/compare/v4.4.14...v4.4.15 [4.4.14]: https://github.com/oscarotero/Embed/compare/v4.4.13...v4.4.14 [4.4.13]: https://github.com/oscarotero/Embed/compare/v4.4.12...v4.4.13 diff -Nru php-embed-4.4.15/debian/changelog php-embed-4.4.17/debian/changelog --- php-embed-4.4.15/debian/changelog 2025-02-24 19:28:12.000000000 +0100 +++ php-embed-4.4.17/debian/changelog 2025-05-14 08:47:59.000000000 +0200 @@ -1,3 +1,27 @@ +php-embed (4.4.17-1) unstable; urgency=medium + + [ Timothée Jaussoin ] + * Fix the subdomain detection, substr was using a wrong offset length + + [ Oscar Otero ] + * new version + + -- David Prévot <taf...@debian.org> Wed, 14 May 2025 08:47:59 +0200 + +php-embed (4.4.16-1) unstable; urgency=medium + + [ Dieter Holvoet ] + * Only match exact main and subdomains + * Stop using regex + + [ Oscar Otero ] + * new version + + [ David Prévot ] + * Update Standards-Version to 4.7.2 + + -- David Prévot <taf...@debian.org> Sun, 11 May 2025 09:06:24 +0200 + php-embed (4.4.15-2) unstable; urgency=medium * Modernize PHPUnit syntax diff -Nru php-embed-4.4.15/debian/control php-embed-4.4.17/debian/control --- php-embed-4.4.15/debian/control 2025-02-24 19:14:58.000000000 +0100 +++ php-embed-4.4.17/debian/control 2025-05-11 09:05:00.000000000 +0200 @@ -20,7 +20,7 @@ php-xml, phpab, phpunit -Standards-Version: 4.7.1 +Standards-Version: 4.7.2 Homepage: https://github.com/oscarotero/Embed Vcs-Git: https://salsa.debian.org/php-team/pear/php-embed.git Vcs-Browser: https://salsa.debian.org/php-team/pear/php-embed diff -Nru php-embed-4.4.15/src/ExtractorFactory.php php-embed-4.4.17/src/ExtractorFactory.php --- php-embed-4.4.15/src/ExtractorFactory.php 2025-01-02 17:53:09.000000000 +0100 +++ php-embed-4.4.17/src/ExtractorFactory.php 2025-05-13 14:42:29.000000000 +0200 @@ -46,7 +46,14 @@ $class = $this->default; foreach ($this->adapters as $adapterHost => $adapter) { - if (substr($host, -strlen($adapterHost)) === $adapterHost) { + // Check if $host is the same domain as $adapterHost. + if ($host === $adapterHost) { + $class = $adapter; + break; + } + + // Check if $host is a subdomain of $adapterHost. + if (substr($host, -strlen($adapterHost) - 1) === ".{$adapterHost}") { $class = $adapter; break; }