-
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
TS-2237: Add unit tests for escapify_url and pure_escapify_url #913
Conversation
FreeBSD build failed! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/600/ for details. |
Linux build failed! See https://ci.trafficserver.apache.org/job/Github-Linux/496/ for details. |
CI builds fail because the functions are not in the code yet. |
[approve ci] |
Linux build failed! See https://ci.trafficserver.apache.org/job/Github-Linux/619/ for details. |
FreeBSD build failed! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/723/ for details. |
FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/744/ for details. |
Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/640/ for details. |
Rebased to obtain the functions from master. |
Looks good to me. |
This addresses two leaks discovered while testing gRPC: one in which the pre-trailer tunnel has to be deallocated when the trailing tunnel is set up, the other in which the header_block memory for the initial headers has to be cleaned up when the new trailing headers are being processed. (cherry picked from commit f487c1f) Conflicts: proxy/http/HttpSM.cc
Unit tests for #866