-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
[2.6 beta1 w/ dco]Remove link-mtu warning. #191
Comments
Looks like it's internally auto-generated [src/openvpn/options.c]. I think the solution is removing this option as soon as possible as 2.6 is still in early beta? |
It's directly affecting 7 src files... Wonder what's the point of this at the beginning |
We need more information of your configuration. Normally if all is identical we should have not have these warnings. Btw: Just because you do not see point or where the warning come from, does not mean that they are/were pointless. Your wording feels extremely agressive/insulting to me. |
yes weird indeed, wait. I didn't say they are/were pointless, where the impression comes from |
client:
|
server:
|
where's that 3 bytes discrepancy coming from... |
Please don't use the bug tracker like a chat. There is no need to split your one message into four individual ones. Also please provide a |
Probably, I'll test it later. |
This repository is only for reporting bug. If you want to discuss, there is http://forum.openvpn.net |
@Originalimoc I am still waiting for your log files. |
Next workday. |
Closing this issue as the submitter is unwilling to cooperate and providing logfiles. If someone can provide log files for this issue I am happy to reopen to investigate this ticket. |
I'm reopening this, as we are still interested to find a good solution for the With the new code, this is easily triggered by having identical ("Just remove the warning" would fix things for 2.6, but clients connecting with an older version would still print the warning, but maybe there is no way we can make this work in a satisfying way. We could remove the warning from the 2.5 codebase as well, of course...) |
The warning was effectively removed from 2.6 by changing the log level so it only appears if you really want to see it (via commit bfd0ef3). 2.5 to 2.6 will still log this on the 2.5 side. |
Identical
--cipher
/--data-cipher AES-128-GCM
/--auth SHA256
/--data-cipher-fallback
/compression no
setting among server and clients, no link-mtu config.Client still log:
WARNING: 'link-mtu' is used inconsistently, local='link-mtu 1516', remote='link-mtu 1513'
Lack of documentation on how it's computed, and no real effect, DEPRECATED anyway, just remove it.
The text was updated successfully, but these errors were encountered: