trafficserver icon indicating copy to clipboard operation
trafficserver copied to clipboard

Q: Whether forward proxy check url

Open oceanwalker opened this issue 3 years ago • 5 comments

Recently found CVE-2021-44040 vulnerabilities, I wonder if forward proxy check url and involved this vulnerabilities.

I review the commits between 8.1.3 and 8.1.4, I found only this config related to vulnerabilities, so please help me, thks! .. ts:cv:: CONFIG proxy.config.http.strict_uri_parsing INT 0 .. ts:cv:: CONFIG proxy.config.http.strict_uri_parsing INT 2

oceanwalker avatar Jul 01 '22 14:07 oceanwalker

Yes that change was in relation to that CVE, the new default should address it AFAIK

ezelkow1 avatar Jul 02 '22 01:07 ezelkow1

First, I try to reproduce the attack, add some blank and illegal characters to the URL, then through forward proxy transmit the request, but ATS didn't reject this request.

Second, I just use forward proxy ability, and didn't use reverse proxy, so I wonder if this is impacted by this CVE?

this file is my records.config records.config.txt

oceanwalker avatar Jul 02 '22 03:07 oceanwalker

@oceanwalker Can you please send an email to [email protected] on how you are trying to reproduce this issue?

bryancall avatar Jul 08 '22 16:07 bryancall

image

无标题

Based on the version 8.1.4 and used Xshell as SSH client, after adding invalid and blank characters shown in the above screenshot, the request is still forwarded, which is contrary to our expectation.

oceanwalker avatar Jul 11 '22 07:07 oceanwalker

@bryancall Considering it is so important to me and a lot of work would based on your answer, could you please share with me about the latest progress as early as you could. Thank you so much!

oceanwalker avatar Jul 18 '22 02:07 oceanwalker

This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

github-actions[bot] avatar Jul 19 '23 02:07 github-actions[bot]