NotificationsYou must be signed in to change notification settings
Fork9.7k
Star30.8k

[HttpClient] Fix regex bearer#35530

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

Jump to bottom

Merged

nicolas-grekas merged 1 commit intosymfony:4.3fromnoniagriconomie:fix-34919

Jan 31, 2020

Merged

[HttpClient] Fix regex bearer#35530

nicolas-grekas merged 1 commit intosymfony:4.3fromnoniagriconomie:fix-34919

Jan 31, 2020

Conversation

Copy link

Contributor

noniagriconomie commentedJan 30, 2020

Q	A
Branch?	4.3
Bug fix?	yes
New feature?	no
Deprecations?	no
Tickets	Fix#34919
License	MIT
Doc PR	none

Small fix related to#34919 (comment)

[HttpClient] Fix regex bearer

cd0db78

carsonbot added Status: Needs Review HttpClient Bug labels

Jan 30, 2020

noniagriconomie requested a review fromnicolas-grekas

January 30, 2020 16:53

Copy link

Member

dunglas commentedJan 30, 2020

Could you also add an unit test to prevent regressions please?

nicolas-grekas added this to the4.3 milestone

Jan 31, 2020

Copy link

Member

nicolas-grekas commentedJan 31, 2020

Thank you@noniagriconomie.

nicolas-grekas added a commit that referenced this pull request

Jan 31, 2020

bug#35530[HttpClient] Fix regex bearer (noniagriconomie)

e39d66e

This PR was merged into the 4.3 branch.Discussion----------[HttpClient] Fix regex bearer| Q             | A| ------------- | ---| Branch?       | 4.3| Bug fix?      | yes| New feature?  | no| Deprecations? | no| Tickets       |Fix#34919| License       | MIT| Doc PR        | noneSmall fix related to#34919 (comment)Commits-------cd0db78 [HttpClient] Fix regex bearer