-
Notifications
You must be signed in to change notification settings - Fork 780
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
Modify TSHttpTxnHookAdd to add hook only once #1320
Conversation
Change-Id: I45c9cdbe305e8b3d186beafb87ab5f8c5b03b757
Marking this for 7.1.0, we really need this in. |
[approve ci] |
Ship it. |
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.
+1
Linux build failed! See https://ci.trafficserver.apache.org/job/Github-Linux/1231/ for details. |
@meeramn Can you re-run clang-format and update the PR please? |
FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1337/ for details. |
Modify TSHttpTxnHookAdd to add hook only once. Change-Id: I737ca666cc05f2e2e2006cbef7f6b5a68b0eb71f
[approve ci] |
Ran clang-format and pushed the change. |
FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1349/ for details. |
Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/1242/ for details. |
FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1350/ for details. |
Well, dammit, I screwed up, this really shouldn't have been merged, you really must squash multiple commits like this into one commit (now I landed one commit that is not correct). :-/ |
Not going to do the reverts here, but please be careful going forward with squashing commits that are fixes / review comments. |
Change-Id: I45c9cdbe305e8b3d186beafb87ab5f8c5b03b757