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

Switch to Building obfs4proxy from Go Source Due to Its Removal from Alpine Repository #57

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

beshkenadze
Copy link

Implemented a build of obfs4proxy from its Go package, as it has been removed from the Alpine repository.

Screenshot

Updates the ReadME with a usage guide. Cleaning up the Dockerfile.
Copy link
Owner

@PeterDaveHello PeterDaveHello left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any chance that we can make this PR not so huge or complex to achieve the same goal?

@beshkenadze
Copy link
Author

Is there any chance that we can make this PR not so huge or complex to achieve the same goal?

What do you mean?
Building an obfs4 binary? That's a major change. We can only keep it.

Or the ability to extend the configuration? I think this is a very useful thing. Then we can configure not only bridges without touching the main configuration. And the changes are very minimal.

It is possible to remove the example config, but it is quite important, because the syntax has changed a bit and there will be a lot of questions why it doesn't work.

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.

None yet

2 participants