<div dir="ltr"><div dir="ltr"><div dir="ltr">Hello lovely WireGuard people,<br><br>I wrote a tiny patch to add a name attribute to peers because identifying<br>clients on my VPN just got very hard. Also hacking on wg source was a lot of<br>fun. I asked @zx2c4 about this a while ago and he was open to the idea.<br><br>It's not _really_ ready for a submission yet because of a few rough edges but<br>I'd love to get some eyes on this if anyone is kind! I've never written any<br>kernel code before and my C is a bit rusty, so please feel free to be picky<br>about the review. I've added '*NOTE:*'s for the reviewer where I'm not sure of<br>the approach.<br><br>A few other questions,<br><br>1. There are barely any inline code comments, is that intentional or something<br>that the community would love to improve? I don't mean the really obvious ones<br>but things that are tricky.<br><br>2. Is there any interest in a getting started guide for developers wanting to<br>work on wireguard itself? Something like the usual CONTRIBUTING.md? I had to dig<br>through a lot of netlink tutorials and old kernel newbie guides to make any<br>progress. It would be helpful to have it in one small doc.<br><br>The patch stays in the `name` branch here[1] and this commit does most of the<br>work[2].<br><br>[1]: <a href="https://github.com/jaseemabid/WireGuard/tree/name">https://github.com/jaseemabid/WireGuard/tree/name</a><br>[2]: <a href="https://github.com/jaseemabid/WireGuard/commit/544e59b4">https://github.com/jaseemabid/WireGuard/commit/544e59b4</a><br><br><div>-- <br><div dir="ltr" class="gmail_signature">Regards, <br><br>Jaseem Abid<br><a href="https://twitter.com/jaseemabid" target="_blank">https://twitter.com/jaseemabid</a><br></div></div></div></div></div>