tarampampam / 3proxy-docker

:calling: Docker image with 3proxy - Tiny free proxy server
https://hub.docker.com/r/tarampampam/3proxy/
Do What The F*ck You Want To Public License
141 stars 39 forks source link

Feature request: Add proxy allow without authentication #47

Closed mjtrangoni closed 7 months ago

mjtrangoni commented 8 months ago

Hi @tarampampam,

First of all, thank you for this project!

I would like to build something like,

env:
WEBPROXYALLOW="* * *api.letsencrypt.org,*.azure.com,*.microsoftonline.com"
WEBPROXYDENY="*"

resulting to a 3proxy.cfg like this,

allow * * *api.letsencrypt.org,*.azure.com,*.microsoftonline.com
deny *

as this currently not possible, I would like to ask you what would be your preferred way of building it? Should we build it in the authentication part or just make it separately afterwards?

Thank you!

tarampampam commented 7 months ago

Hello @mjtrangoni! I've added support for the additional environment variable named EXTRA_CONFIG (commit: 4f60889624f3f038772923876ee7086e51e18b1e), so now you should be able to customize the configuration according to your needs!

Release: https://github.com/tarampampam/3proxy-docker/releases/tag/v1.9.0

mjtrangoni commented 7 months ago

@tarampampam I gave the new image a try, and it seems that order is critical here. As you can see, EXTRA_CONFIG comes at the very end, and it does not allow nor block anything at all. My proposal would be to move it before the proxy line.

Thank you!