Closed accwebs closed 3 years ago
All good catches and call outs, thank you! I'll tweak, squash, and re-sign as a new commit (unless you prefer I wait and squash the history later).
Should be updated. Once you're good with the changes, let me know and I'll squash the history down.
Should be updated. Once you're good with the changes, let me know and I'll squash the history down.
There's no need for you to squash the commits down; I can do this as part of the merge.
Thanks very much for the patch; I'll get it merged now.
Thanks!
In the case of trying to forward broadcast packets over tun-based VPNs, the tun interfaces unfortunately don't support broadcast packets. A workaround is to rewrite the outbound packet's destination address to be the remote router's IP address. And then that router would rewrite the packet's destination address to the broadcast address.
Admittedly I'm kinda a noob at C programming but I hope I did this right?