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
Appearance settings

[Notifier] [5.3] Make sure Http TransportException is not leaking #42185

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 19, 2021
Merged

[Notifier] [5.3] Make sure Http TransportException is not leaking #42185

merged 1 commit into from
Jul 19, 2021

Conversation

Nyholm
Copy link
Member

@Nyholm Nyholm commented Jul 19, 2021

Q A
Branch? 5.3
Bug fix? yes
New feature? no
Deprecations? no
Tickets
License MIT
Doc PR

Same as #42182 but for bridges introduced in 5.3

@Nyholm Nyholm requested a review from OskarStark as a code owner July 19, 2021 01:01
@carsonbot carsonbot added this to the 5.3 milestone Jul 19, 2021
@carsonbot carsonbot changed the title [Notifier][5.3] Make sure Http TransportException is not leaking [Notifier] [5.3] Make sure Http TransportException is not leaking Jul 19, 2021
@OskarStark OskarStark force-pushed the 5.3-notifier-http-transport-exception branch from 4949c54 to 77b2409 Compare July 19, 2021 05:58
@OskarStark
Copy link
Contributor

Good catch, thanks Tobias.

@OskarStark OskarStark merged commit a2f394e into symfony:5.3 Jul 19, 2021
@fabpot fabpot mentioned this pull request Jul 26, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@OskarStark OskarStark OskarStark approved these changes

Assignees
No one assigned
Labels
Bug Notifier Status: Reviewed
Projects
None yet
Milestone
5.3
Development

Successfully merging this pull request may close these issues.
3 participants
@Nyholm @OskarStark @carsonbot
Morty Proxy This is a proxified and sanitized view of the page, visit original site.