Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[HttpClient] Fix Undefined array key "connection"#59044

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
PhilETaylor wants to merge10,000 commits intosymfony:5.4fromPhilETaylor:59039-undefinedarraykey

Conversation

PhilETaylor
Copy link
Contributor

QA
Branch?7.2
Bug fix?yes
New feature?no
Deprecations?no
IssuesFix#59039
LicenseMIT

Check for array key before attempting to access it
Code style for human reading

dunglasand others added30 commitsNovember 6, 2024 08:47
… default domain node visitor (xabbuh)This PR was merged into the 7.2 branch.Discussion----------[TwigBridge] use reproducible variable names in the default domain node visitor| Q             | A| ------------- | ---| Branch?       | 7.2| Bug fix?      | no| New feature?  | yes| Deprecations? | no| Issues        |Fixsymfony#57609 (comment)| License       | MITCommits-------500d72f use reproducible variable names in the default domain node visitor
* 5.4:  [DoctrineBridge] Backportsymfony#53681
* 6.4:  [DoctrineBridge] Backportsymfony#53681
* 7.1:  [DoctrineBridge] Backportsymfony#53681
…RIs that contain invalid characters (nicolas-grekas)This PR was merged into the 7.2 branch.Discussion----------[DependencyInjection][HttpClient][Routing] Reject URIs that contain invalid characters| Q             | A| ------------- | ---| Branch?       | 7.2| Bug fix?      | no| New feature?  | no| Deprecations? | no| Issues        | -| License       | MITThe behavior of `parse_url()` doesn't match how browsers parse URLs. This PR ensures we won't accept URLs that are invalid perhttps://url.spec.whatwg.org/:- URLs that contain a backslash- or start/end with a control-char or a space- or contain a CR/LF/TAB characterCommits-------d73003e [DependencyInjection][Routing][HttpClient] Reject URIs that contain invalid characters
* 5.4:  [Runtime] fix tests
* 6.4:  [Runtime] fix tests
* 7.1:  [HttpFoundation] Fix merge  [Runtime] fix tests
* 5.4:  [Process] Fix test
* 6.4:  [Process] Fix test
* 7.1:  [Process] Fix test
* 5.4:  Port Appveyor to GitHub Actions  normalize paths to avoid failures if a path is referenced by different names  skip autocomplete test when stty is not available  Bump Symfony version to 5.4.47  Update VERSION for 5.4.46  Update CONTRIBUTORS for 5.4.46  Update CHANGELOG for 5.4.46
* 6.4:  Port Appveyor to GitHub Actions  normalize paths to avoid failures if a path is referenced by different names  skip autocomplete test when stty is not available  Bump Symfony version to 6.4.15  Update VERSION for 6.4.14  Update CHANGELOG for 6.4.14  Bump Symfony version to 5.4.47  Update VERSION for 5.4.46  Update CONTRIBUTORS for 5.4.46  Update CHANGELOG for 5.4.46
fabpotand others added11 commitsNovember 27, 2024 13:55
* 5.4:  [HttpClient] Fix streaming and redirecting with NoPrivateNetworkHttpClient  Bump Symfony version to 5.4.49  Update VERSION for 5.4.48  Update CONTRIBUTORS for 5.4.48  Update CHANGELOG for 5.4.48
* 6.4:  [HttpClient] Fix streaming and redirecting with NoPrivateNetworkHttpClient  Bump Symfony version to 6.4.17  Update VERSION for 6.4.16  Update CHANGELOG for 6.4.16  Bump Symfony version to 5.4.49  Update VERSION for 5.4.48  Update CONTRIBUTORS for 5.4.48  Update CHANGELOG for 5.4.48
* 7.1:  [HttpClient] Fix streaming and redirecting with NoPrivateNetworkHttpClient  Bump Symfony version to 7.1.10  Update VERSION for 7.1.9  Update CHANGELOG for 7.1.9  Bump Symfony version to 6.4.17  Update VERSION for 6.4.16  Update CHANGELOG for 6.4.16  Bump Symfony version to 5.4.49  Update VERSION for 5.4.48  Update CONTRIBUTORS for 5.4.48  Update CHANGELOG for 5.4.48
Signed-off-by: Phil E. Taylor <phil@phil-taylor.com>
@discordier
Copy link
Contributor

Must target against 5.4 or backport - original PR was against 5.4.

@PhilETaylor
Copy link
ContributorAuthor

ah crap, changing the base screwed the PR :) Fridays suck. I suck. Will redo.

PhilETaylor added a commit to PhilETaylor/symfony that referenced this pull requestNov 29, 2024
Signed-off-by: Phil E. Taylor <phil@phil-taylor.com>
@PhilETaylor
Copy link
ContributorAuthor

5.4 based replacement here#59046

nicolas-grekas pushed a commit to PhilETaylor/symfony that referenced this pull requestJan 8, 2025
Signed-off-by: Phil E. Taylor <phil@phil-taylor.com>
nicolas-grekas added a commit that referenced this pull requestJan 8, 2025
…ylor)This PR was submitted for the 5.4 branch but it was merged into the 6.4 branch instead.Discussion----------[HttpClient] Fix Undefined array key `connection`| Q             | A| ------------- | ---| Branch?       | 5.4| Bug fix?      | yes| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->| Issues        |Fix#59039 <!-- prefix each issue number with "Fix #", no need to create an issue if none exists, explain below instead -->| License       | MITCheck for array key before attempting to access itCode style for human readingCommits-------51c32ef [HttpClient] Fix Undefined array key "connection"#59044
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@lyrixxlyrixxAwaiting requested review from lyrixxlyrixx is a code owner

@ycerutoycerutoAwaiting requested review from ycerutoyceruto is a code owner

@chalasrchalasrAwaiting requested review from chalasrchalasr is a code owner

@dunglasdunglasAwaiting requested review from dunglasdunglas is a code owner

@xabbuhxabbuhAwaiting requested review from xabbuhxabbuh is a code owner

Assignees
No one assigned
Projects
None yet
Milestone
7.2
Development

Successfully merging this pull request may close these issues.

29 participants
@PhilETaylor@discordier@carsonbot@dunglas@xabbuh@nicolas-grekas@fabpot@matyo91@derrabus@stof@94noni@janedbal@0xb4lint@GregRbs92@smnandre@mtarld@KurtThiemann@HypeMC@StefanoTarditi@BartBaaten@zanbaldwin@alexislefebvre@keulinho@wanxiangchwng@kbond@keradus@neodevcode@alexandre-daubois@wouterj

[8]ページ先頭

©2009-2025 Movatter.jp