Skip to content

[HttpFoundation] Remove possibility to pass null as $requestIp in IpUtils #43550

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 18, 2021

Conversation

W0rma
Copy link
Contributor

@W0rma W0rma commented Oct 17, 2021

Q A
Branch? 6.0
Bug fix? no
New feature? no
Deprecations? no
Tickets #43350 (comment)
License MIT
Doc PR

Removes the code which was deprecated in #43411

@fabpot
Copy link
Member

fabpot commented Oct 18, 2021

Thank you @W0rma.

@fabpot fabpot merged commit be8401b into symfony:6.0 Oct 18, 2021
@fabpot fabpot mentioned this pull request Nov 5, 2021
@W0rma W0rma deleted the ip-utils-request-ip-not-nullable branch November 7, 2021 10:14
nicolas-grekas added a commit that referenced this pull request Jun 30, 2023
…onTrait` where it is not used (GromNaN)

This PR was merged into the 6.4 branch.

Discussion
----------

[DoctrineBridge][HttpFoundation] Remove `ExpectDeprecationTrait` where it is not used

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | n/a

This unused code has been left since 6.0 by #43100 and #43550.
The PR targets 6.4 because this is not a bug, but minor code cleanup.

Commits
-------

9317cbd Remove ExpectDeprecationTrait where it is not used
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants