Bump symfony/dom-crawler from 7.4.6 to 8.0.12#173
Conversation
Bumps [symfony/dom-crawler](https://github.com/symfony/dom-crawler) from 7.4.6 to 8.0.12. - [Release notes](https://github.com/symfony/dom-crawler/releases) - [Changelog](https://github.com/symfony/dom-crawler/blob/8.1/CHANGELOG.md) - [Commits](symfony/dom-crawler@v7.4.6...v8.0.12) --- updated-dependencies: - dependency-name: symfony/dom-crawler dependency-version: 8.0.12 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
Bot
added
dependencies
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit e9769a4. Configure here.
| "symfony/deprecation-contracts": "^2.5|^3", | ||
| "symfony/polyfill-ctype": "~1.8", | ||
| "symfony/polyfill-mbstring": "~1.0" | ||
| "php": ">=8.4", |
There was a problem hiding this comment.
Major version bump requires PHP 8.4, CI uses 8.3
High Severity
The bumped symfony/dom-crawler v8.0.12 requires "php": ">=8.4", but both CI workflows (.github/workflows/php.yml and release-version.yml) are configured with php-version: 8.3. The composer install step in the Quality workflow will fail because PHP 8.3 does not satisfy the >=8.4 constraint. This is a major version bump (7.x → 8.x) that raises the minimum PHP requirement and will break CI.
Reviewed by Cursor Bugbot for commit e9769a4. Configure here.
Bumps symfony/dom-crawler from 7.4.6 to 8.0.12.
Release notes
Sourced from symfony/dom-crawler's releases.
... (truncated)
Changelog
Sourced from symfony/dom-crawler's changelog.
... (truncated)
Commits
011b0ceMerge branch '7.4' into 8.0b59b591Merge branch '6.4' into 7.47e65f76Merge branch '5.4' into 6.40366e1eMerge branch '7.4' into 8.0b18373eMerge branch '6.4' into 7.4505deba[DomCrawler] FixChoiceFormField::addChoice()clobbering values on multi-se...fe2f2b6Merge branch '7.4' into 8.0f474100Update XSD references in phpunit.xml.dist filesb4cf17f[DomCrawler] Fix XXE in addXmlContent() by not enablingvalidateOnParse6d86f97[Tests] Fix "Incomplete version" PHPUnit warningsDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)You can disable automated security fix PRs for this repo from the Security Alerts page.
Note
Medium Risk
Major DomCrawler upgrade with PHP >= 8.4 enforced in the lockfile and XML parsing/security behavior changes; impact is mostly on dev/test tooling unless production installs the same lock.
Overview
Updates
composer.lockto pullsymfony/dom-crawlerfrom 7.4.6 to 8.0.12 (Symfony 8 major), along with related Symfony packages (deprecation-contracts,polyfill-ctype,polyfill-mbstring,polyfill-php80). The bump is transitive (e.g. Codeception /symfony/browser-kit);composer.jsonis unchanged.8.x drops the
masterminds/html5dependency and raises the package’s PHP requirement to >= 8.4; HTML parsing shifts to PHP’s native HTML5 parser and the optional$useHtml5Parserconstructor flag is removed. Release notes for this line include an XXE fix inaddXmlContent()and fixes around multi-select form fields.Reviewers should confirm local CI and dev environments run PHP 8.4+ before merging, since older PHP versions will no longer satisfy the locked tree.
Reviewed by Cursor Bugbot for commit e9769a4. Bugbot is set up for automated code reviews on this repo. Configure here.