Re: [Openvpn-devel] Summary of the community meeting (Wed, 28th Nov 2018)

2018-11-30 Thread Jan Just Keijser
On 30/11/18 17:27, Kristian McColm wrote: 1500 is not safe on many mobile/cell networks as most have a smaller MTU like 1460 due to GTP headers used between the PGW and eNodeB. There is also reduced MTU when performing NAT64, i.e. packets translated from IPv4 to IPv6 become 20 bytes larger aft

Re: [Openvpn-devel] Summary of the community meeting (Wed, 28th Nov 2018)

2018-11-30 Thread Kristian McColm
1500 is not safe on many mobile/cell networks as most have a smaller MTU like 1460 due to GTP headers used between the PGW and eNodeB. There is also reduced MTU when performing NAT64, i.e. packets translated from IPv4 to IPv6 become 20 bytes larger after translation due to additional IP header b

Re: [Openvpn-devel] Summary of the community meeting (Wed, 28th Nov 2018)

2018-11-30 Thread Jan Just Keijser
Hi Lev, Simon, On 30/11/18 07:10, Simon Matter wrote: Hi Jan Just, (forgot to add openvpn-devel in previous mail) Some background information. In openvpn3 we decided not to implement fragments, because: - this is quite a big feature which has to be supported through the whole stack (client

Re: [Openvpn-devel] [PATCH v5] Implement block-ipv6

2018-11-30 Thread Gert Doering
Hi, On Mon, Oct 29, 2018 at 06:20:40PM +0100, Arne Schwabe wrote: > This can be used to redirect all IPv6 traffic to the tun interface, > effectively black holing the IPv6 traffic. Without ICMPv6 error > messages this will result in timeouts when the server does not send > error codes. block-ipv6

[Openvpn-devel] [PATCH 3/3] uncrustify openvpn/ sources

2018-11-30 Thread Gert Doering
There are an amazing number of brackets that were either totally missing, or have snuck up on the "for(...){" line. Further, uncrustify wants "|" in multi-line logical expressions now at the beginning of the new line, and "PRIi64" now gets surrounding spaces. Added "sp_after_semi_for_empty=Add" t

[Openvpn-devel] [PATCH 2/3] uncrustify openvpnserv/ sources

2018-11-30 Thread Gert Doering
this is really just whitespace changes, but will make running uncrustify as pre-commit-check easier if the "base sources" won't see changes Signed-off-by: Gert Doering --- src/openvpnserv/interactive.c | 4 ++-- src/openvpnserv/service.c | 4 ++-- src/openvpnserv/service.h | 2 ++ src/op

[Openvpn-devel] [PATCH 1/3] Uncrustify sample-plugin sources according to code style

2018-11-30 Thread Gert Doering
Signed-off-by: Gert Doering --- .../keying-material-exporter-demo/keyingmaterialexporter.c | 6 -- sample/sample-plugins/log/log.c | 4 sample/sample-plugins/log/log_v3.c | 4 3 files changed, 12 insertions(+), 2 deletions(-) di