-
-
Notifications
You must be signed in to change notification settings - Fork 360
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
[FEAT / BUG] <Modifiable 'ListenPort' used by host/wireguard incoming connections through docker configuration> #319
Comments
Thanks for opening your first issue here! Be sure to follow the relevant issue templates, or risk having this issue marked as invalid. |
Why should it? It's called
Map the ports however you like: |
I think there’s a misinterpretation here.
Example: Assume one sets mapping as
I agree totally, the |
No. Container port is always |
Right, so it seems it’d be logical to have to automatically have the program set the
Do you mean the container port will remain |
No offense, but you need to read up on the general concept of docker port mapping to get up to speed.
No. They don't need to match. I don't know how else to explain it to you other than what I wrote above.
That's how docker port mapping works. That's the point of it. Container port remains the same, host port can be anything you want.
You wouldn't do that. You would do You really need to read up more. I don't have the bandwidth to continue this discussion any longer. |
I'm familiar with how port mapping using docker works.
Host So it shouldn’t be hard coded to It’s an issue that has come up plenty of times and still hasn’t gotten a clear fix or documentation explanation. If the program doesn’t want to work in an automated fashion if a user changes the mapped container port then either; specify “DO NOT CHANGE” in your documentation, or make the server Is that fair, or is your answer still “no, the program will not work when used this way. you should know this port mapping configuration is a requirement” while this requirement it’s not stated once in documentation, or clearly in your responses .. ? In #51 you do say :
So, my suggestion is that the field in question should be generated to match this port. |
if I have understood this point correctly, you would like to change the container port. generally speaking this field is set to a value that the internal application inside the container is listening on. It should never need to be changed. That is what the host port is for. For those who run multiple instances of the WireGuard container, it is sufficient to change only the host port mapping to avoid port conflicts (assuming each container is running in a bridge network mode).
The idea of not changing the container-side port value in Port |
Is this a new feature request?
Wanted change
Either A. An environment variable that will modify 'ListenPort' field in the generated host wireguard config (default wg_confs/wg0.conf)
Or B. Have server automatically set 'ListenPort' to match wireguard host port set in docker configuration.
Reason for change
First and foremost, the attitude that @aptalca gives users submitting valid issues/requests is unjustified.
Half a dozen requests about the same issue and his responses remain equally unhelpful yet increasingly more obnoxious.
As he mentions in #176 the readme is clear, yes, but does not mention a single thing about why the 'ListenPort' set in generated server wireguard config doesn't / shouldn't match the docker specified host port used for wireguard.
Next, its stated in #175 that "There is also no reason for you to change [the listen port]". Completely untrue, for example, in the instance that a previous wireguard configuration on system or other program is using port 51820.
Having multiple simultaneous wireguard configurations using different ports on a single client or even host has it's use cases. One shouldn't have to modify a field in an autogenerated and undocumented fine in order to allow this to work.
Proposed code change
No response
The text was updated successfully, but these errors were encountered: