Replace netsh for handling MTU#17
Conversation
|
Fixed the clippy warnings here: #18 |
9d48474 to
270aade
Compare
|
Looks like the version needs to be bumped. Do you want me to do that? |
|
Needn't, since 'stdcall' is 'system` in windows. I have no time to review it right now, maybe tomarrow. |
|
Do you think you could have a look at this? |
|
I apologize for the long delay, as I encountered numerous setbacks with setting the MTU in Wintun. This is due to a flaw in the Wintun kernel driver. The MTU settings weren't actually working, but many disagreed with me. To quell the debate, I agreed with these opinions and pretended that the MTU modifications were effective. On the other hand, I don't trust the various dazzling APIs provided by Microsoft for setting network parameters, as I've suffered countless losses using them. Therefore, I prefer to use Microsoft's command-line tools like Did your modifications to these MTU settings actually work? If you have thoroughly verified them, they do indeed work and have been proven in practice. I would be happy to merge this PR. |
|
No worries. I've done some tests below. The MTU is correct according to this: Also, running the I could also add that the Mullvad VPN app has been using |
Also, use the simpler
GetIpInterfaceEntryfor obtaining the interface MTU. This should reliably work.