Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build docker image #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

build docker image #1

wants to merge 1 commit into from

Conversation

asteny
Copy link

@asteny asteny commented Apr 16, 2018

I built docker image and added some description. This code very useful in Russia now^ because our government blocking telegram.org

@matt-snider
Copy link
Owner

Hi @asteny

Thank you for your contribution 😄!

It looks good except for a few small things:

  • The default port for socks5 should be 1080
    • For docker, this can be the internal port and the user can still do -p 1488:1080 if port1488 is desired
  • All the docker-specific files should go in a docker subdirectory to keep things organized

I'm not especially familiar with the use case here for telegram. So just a few questions regarding that:

  • What is this url tg://socks?server=ip_or_dns_name&port=port&user=username&pass=youpassword? It looks like something telegram specific, but I couldn't find any documentation on it or what format it should follow
  • Why port 1488? Is this also something telegram-specific?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants