-
-
Notifications
You must be signed in to change notification settings - Fork 588
[Plugin] request.path
always None
in v2.4.0rc7
#1035
-
Since updating to v2.4.0.rc7, request.path is always set to None:
|
Beta Was this translation helpful? Give feedback.
All reactions
Replies: 1 comment · 9 replies
-
Please provide a concrete reproducible example. Example, whats the URL, how can we reproduce it using curl, or may be how does the raw request packet looks like? We have integration and unit test, something like this cannot be missed. Kindly double check your code. |
Beta Was this translation helpful? Give feedback.
All reactions
-
I would love to see some raw data to understand this problem. Here is how you folks can help:
|
Beta Was this translation helpful? Give feedback.
All reactions
-
here is my curl
here are outputs after uncomment codes above:
plugin code : |
Beta Was this translation helpful? Give feedback.
All reactions
-
So IIUC, you are getting path as |
Beta Was this translation helpful? Give feedback.
All reactions
-
Yes ,so what's the problem? tks I found that when I request with http:
https:
|
Beta Was this translation helpful? Give feedback.
All reactions
-
https://github.com/abhinavsingh/proxy.py/blob/develop/proxy/http/parser/parser.py#L460 Path comes from here. Do read the related I'll also recommend to read https://github.com/abhinavsingh/proxy.py/blob/develop/tutorial/http_parser.ipynb which will explain inner working of parser class. |
Beta Was this translation helpful? Give feedback.
All reactions
This discussion was converted from issue #1034 on January 21, 2022 17:10.
Please provide a concrete reproducible example. Example, whats the URL, how can we reproduce it using curl, or may be how does the raw request packet looks like?
We have integration and unit test, something like this cannot be missed.
Kindly double check your code.