swagger-api / swagger-api/swagger-codegen
[Python] generator doesn't support SOCKS proxy
Nobody has claimed this yet.
- Dominant language
- Mustache
- Stars
- 17.8k
- Forks
- 6k
- PR merge metrics
- No merged PRs in 30d
Description
Description
Python generation doesn't include a SOCKS proxy connection manager in the rest connector
Swagger-codegen version
2.4.0
Suggest a fix/enhancement
In the generated python code, rest.py in the proxy configuration switch it should detect if the proxy url starts with "socks" and switch between SOCKSProxyManager() and ProxyManager(). A good example is in the Python library "requests".
requests.adapters.HTTPAdapter().proxy_manager_for()
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the generated Python rest.py proxy configuration and compare its behavior with requests.adapters.HTTPAdapter().proxy_manager_for(). Trace where the Python generator's rest.py template is defined, then verify that proxy URLs beginning with "socks" select SOCKSProxyManager while other proxy URLs select ProxyManager.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100