Bump the composer group across 1 directory with 3 updates - #1
Bump the composer group across 1 directory with 3 updates#1dependabot[bot] wants to merge 1 commit into
Conversation
Bumps the composer group with 2 updates in the / directory: [guzzlehttp/guzzle](https://github.com/guzzle/guzzle) and [league/commonmark](https://github.com/thephpleague/commonmark). Updates `guzzlehttp/guzzle` from 7.7.0 to 7.15.1 - [Release notes](https://github.com/guzzle/guzzle/releases) - [Changelog](https://github.com/guzzle/guzzle/blob/8.0/CHANGELOG.md) - [Commits](guzzle/guzzle@7.7.0...7.15.1) Updates `guzzlehttp/psr7` from 2.5.0 to 2.13.0 - [Release notes](https://github.com/guzzle/psr7/releases) - [Changelog](https://github.com/guzzle/psr7/blob/3.0/CHANGELOG.md) - [Commits](guzzle/psr7@2.5.0...2.13.0) Updates `league/commonmark` from 2.4.0 to 2.8.3 - [Release notes](https://github.com/thephpleague/commonmark/releases) - [Changelog](https://github.com/thephpleague/commonmark/blob/2.8/CHANGELOG.md) - [Commits](thephpleague/commonmark@2.4.0...2.8.3) --- updated-dependencies: - dependency-name: guzzlehttp/guzzle dependency-version: 7.15.1 dependency-type: direct:production dependency-group: composer - dependency-name: guzzlehttp/psr7 dependency-version: 2.13.0 dependency-type: indirect dependency-group: composer - dependency-name: league/commonmark dependency-version: 2.8.3 dependency-type: indirect dependency-group: composer ... Signed-off-by: dependabot[bot] <support@github.com>
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
| Duplication | 0 |
AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.
TIP This summary will be updated as you push new changes.
There was a problem hiding this comment.
Pull Request Overview
While this PR successfully targets updates for Guzzle and Commonmark, it leaves the application exposed to two high-severity security vulnerabilities in symfony/process and laravel/framework. Additionally, the PR lacks automated tests to verify that the upgraded libraries do not break existing HTTP, Markdown rendering, or header normalization logic. Although Codacy rates the PR as up to standards, these unaddressed security flaws and the lack of verification scenarios should be resolved before merging.
About this PR
- There are no new or existing tests included in the PR to validate the impact of these library updates on the application's functionality.
- The PR description summary incorrectly states '2 updates' while the PR actually includes three dependency upgrades (Guzzle, PSR-7, and Commonmark).
2 comments outside of the diff
composer.lock
line 4968🔴 HIGH RISK
The symfony/process component is affected by CVE-2024-51736, which allows command execution hijacking on Windows systems if an executable with the same name as the command is present in the current working directory.
line 1030🔴 HIGH RISK
The laravel/framework package is affected by CVE-2024-52301. In specific PHP-CGI configurations where the register_argc_argv php.ini setting is enabled, the application is vulnerable to command-line argument injection.
Test suggestions
- Missing recommended test scenario: Verify that HTTP client functionality remains correct under Guzzle 7.15.1, particularly regarding the new security-hardened cookie handling and redirect logic.
- Missing recommended test scenario: Verify that Markdown rendering with league/commonmark 2.8.3 correctly handles fenced code blocks and adheres to safe URL filtering rules.
- Missing recommended test scenario: Ensure that the locale-independent ASCII folding introduced in psr7 2.13.0 does not break existing header or host normalization logic.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Missing recommended test scenario: Verify that HTTP client functionality remains correct under Guzzle 7.15.1, particularly regarding the new security-hardened cookie handling and redirect logic.
2. Missing recommended test scenario: Verify that Markdown rendering with league/commonmark 2.8.3 correctly handles fenced code blocks and adheres to safe URL filtering rules.
3. Missing recommended test scenario: Ensure that the locale-independent ASCII folding introduced in psr7 2.13.0 does not break existing header or host normalization logic.
TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback
Bumps the composer group with 2 updates in the / directory: guzzlehttp/guzzle and league/commonmark.
Updates
guzzlehttp/guzzlefrom 7.7.0 to 7.15.1Release notes
Sourced from guzzlehttp/guzzle's releases.
... (truncated)
Changelog
Sourced from guzzlehttp/guzzle's changelog.
... (truncated)
Commits
61443dfRelease 7.15.17b68220Security fixes 7.15 (#3901)90bd104Release 7.15.0e8c879eRequireguzzlehttp/psr7^2.13(#3870)0d578f5Trigger runtime deprecations for legacy functions and methods (#3896)d5fcd1eDeprecate JSON utility methods (#3889)3281fe2Go with"guzzlehttp/test-server": "^0.7"(#3890)ba662c9Remove stale framing from redirected requests (#3881)dd32f84Bound cURL uploads to declared lengths (#3882)a29e961Merge branch '7.14' into 7.15Updates
guzzlehttp/psr7from 2.5.0 to 2.13.0Release notes
Sourced from guzzlehttp/psr7's releases.
... (truncated)
Changelog
Sourced from guzzlehttp/psr7's changelog.
... (truncated)
Commits
dad8962Release 2.13.01bc4f53Trigger a runtime deprecation for Header::normalize() (#866)a1372eaAdd tests locking in no-file upload handling (#856)d9561d1Merge branch '2.12' into 2.139365d57Release 2.12.5b82bac4Add caseless comparison and asciiUcFirst helpers (#854)2cf293eMerge branch '2.12' into 2.13c685419Fold cross-origin host comparison without locale sensitivity (#852)226b5b3Add locale-independent ASCII case folding helpers (#850)1acfa28Use locale-independent ASCII folding for header and host names (#849)Updates
league/commonmarkfrom 2.4.0 to 2.8.3Release notes
Sourced from league/commonmark's releases.
... (truncated)
Changelog
Sourced from league/commonmark's changelog.
... (truncated)
Commits
1902f60Prepare for 2.8.3 release171f927Update changelog51b529fFix tab handling in fenced code blocks (#1130)cfb11ecFix over-blocking of safe URLs by the unsafe link filter (#1131)fb59dbcStrengthen unsafe link and image renderer test assertions (#1129)4d0b5b1Fix typos and broken formatting in the 2.x docs (#1128)73d4f1fAdd Anthropic to the list of sponsors0faf9eeMerge pull request #1125 from thephpleague/fix/markdownlint-md060a85b4adFetch full git history for the Markdownlint jobe6dc715Fix Markdownlint CI job by not overriding DEFAULT_BRANCHDependabot 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 <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditionsYou can disable automated security fix PRs for this repo from the Security Alerts page.