-
Notifications
You must be signed in to change notification settings - Fork 8.3k
Merging #7572 #10547
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
Merging #7572 #10547
Changes from all commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
9dc7054
Add TestCase for feature/ISSUE-5436
TCeason 159ba24
ISSUES-5436 support custom http
zhang2014 183eb82
ISSUES-5436 support custom http [part 2]
zhang2014 60abff3
ISSUES-5436 support custom http [part 3]
zhang2014 fd00757
ISSUES-5436 support custom http [part 4]
zhang2014 7aef95b
ISSUES-5436 support custom http [part 5]
zhang2014 6b716e5
ISSUES-5436 support custom http [part 6]
zhang2014 07ed4ba
ISSUES-5436 add integration test for custom http
zhang2014 847f7ab
ISSUES-5436 fix build failure & fix test failure
zhang2014 dafba9d
ISSUES-5436 add integration test for custom http
TCeason 8105a9b
ISSUES-5436 fix review suggestions
zhang2014 b82eee5
ISSUES-5436 fix review suggestions & add some '?' re2 test
zhang2014 1eda48b
fix bad git rebase
zhang2014 835dc4c
After merge upsream master fix
zhang2014 57cbecf
ISSUES-5436 reworking predefine http
zhang2014 318ab3b
ISSUES-5436 try fix build failure & pvs & style
zhang2014 0070f75
ISSUES-5436 fix integration test failure & add test
zhang2014 8123094
ISSUES-5436 add integration test
zhang2014 d56002b
ISSUES-5436 fix ya make
zhang2014 e72a484
minor fixes
tavplubix 9d43c2f
save compiled regex
tavplubix 5a1d22a
remove redundant configs from test
tavplubix aeac8cb
fix
tavplubix File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CompiledRegexPtr is shared multiple handlers. it's thread safe?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using compiled
RE2from multiple threads is safe. However, we also havere2_stwith disabled internal mutex and there are somemutablefields inRE2, so I'm not sure about safety ofconstmethods ofre2_st::RE2version. I will replace it with ordinaryre2, thanks