Enable using of configured http proxy for media#1298
Conversation
|
Does this allow to toggle on or off in the preferences? |
No. I don't think it should be configurable by user. It's related to http proxy that is used for outgoing http requests, so it's part of nitter server configuration. |
|
Now that I think again about this PR, I didn't fully understand it well. It's not about activating or deactivating the media proxy in nitter. But the http proxy when used in nitter. Sorry. |
|
I'm not sure I agree with this change. The purpose of the proxy is circumvent IP bans, and avoid IP-based detection. API requests are low bandwidth, media certainly isn't, and most proxies are either slow, expensive, or both. If you add a separate |
Currently, configured http proxy (
Config.proxy,Config.proxy_auth) is only used for API calls. Media (twimg.com) is always downloaded directly. This PR adds usage of proxy for media too.This PR adds some code duplication: both
http_poolandroutes/mediamodules now havesetHttpProxymethod andproxyvar. Maybe I should refactor this a bit and make single global var and setter somewhere, but I'm not sure where to put it.Also, maybe there should be separate proxy settings for API and media in config?
This is my first code edits in nim language.