Docker support

[Viktor45]

Added simple dockerfile and manual line

[etidart]

There is no need for cloning the repo, because we already have it with Dockerfile. So, COPY . /opt/byedpi.

[Viktor45]

There is no need for cloning the repo, because we already have it with Dockerfile. So, COPY . /opt/byedpi.

Sometimes users just downloads one Dockerfile at all, instead of full repo. I think Dockerfile can be self-contained, no need to download all repository code to operating systems that don't have git installed, you might not even have a compiler on the host

[ProMix0]

Anyway, fetching specified version always better than fetching latest master

[Viktor45]

Anyway, fetching specified version always better than fetching latest master

Great advice, thanks! Fixed to latest release code.

[hufrea]

For what?

[Viktor45]

For what?

Ok, Removed unnecessary dependencies

build-base + linux headers = build tools + linux/tcp.h
curl + git = getting latest github release code from releases only, not master

[hufrea]

split is meaningless in the same position as disorder (it won't send anything).

[hufrea]

I think this should be moved to dist/

[Viktor45]

I think this should be moved to dist/

moved

[olegshulyakov]

Add debug logs for docker:
--disorder 1 --auto=torst --tlsrec 1+s --debug 1

[Viktor45]

Add debug logs

This is just an example! You need to make your parameters yourself!

[olegshulyakov]

Better split git clone and apk add, see https://github.com/tazihad/byedpi-docker/blob/main/Dockerfile

[Viktor45]

OK

[Saiv46]

@hufrea Any progress?

[inizovtsev]

Guys, did you have any issues running the app inside docker? I tried to do it on mac os using Docker Desktop for Mac with the following dockerfile:

FROM alpine:latest

ARG version=v0.14.1
ARG filename=byedpi-14.1-x86_64.tar.gz

WORKDIR /app

RUN \
    wget https://github.com/hufrea/byedpi/releases/download/${version}/${filename} \
    && tar xzf ${filename}

EXPOSE 1080

CMD ["./ciadpi-x86_64", "--oob", "3"]

and when I built it and ran it (docker run --rm -P -p 0.0.0.0:1080:1080 byedpi:latest) I got the following error in Firefox:

Secure Connection Failed

An error occurred during a connection to www.google.com. Peer received an SSL record that was longer than is permitted.

Error code: SSL_ERROR_RECORD_OVERFLOW_ALERT

This option works perfect for byedpi directly running on mac os (I built it natively) but it does not work inside docker for mac. I think it can be related to docker network layer (I guess it somehow fragments tcp packets or so) but I have no proofs. Other options like --fake did not cause this error but youtube did not work with them on my ISP.

If I'm right and docker network layer (at least for bridge network) is reason for this error then there are some limitations for dockerizing the app.

[Viktor45]

any issues

use issues instead, dont mix -P and -p options.

[inizovtsev]

dont mix -P and -p options.

It happens in any case so looks like it does not matter for docker.

use issues instead

I'll recheck it after the PR is merged and then will create an issue.

[UPDATE]: checked it on vps with ubuntu and did not face with the issue. So it most likely related to docker desktop for mac (I guess due to hypervisor). Wireshark shows there is no tcp packet with URG flag in that case, so tcp data is malformed somehow.