<div dir="ltr"><<<div>I'm very much in favour of this (updating `setconf` to use this new syncronisation approach), if anything it feels more logical and is how I initially (wrongly) assumed `setconf` behaved when starting out with WireGuard a while back.<br></div><div>>></div><div><br></div><div>+1 ... it's better to keep the same command if its definition can be expanded (fewer things to remember and less mental clutter)</div><div><br></div><div>p.s. does this overlap with similar planned in wg-dynamic?</div><div><br></div><div> <br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 11, 2019 at 5:23 PM Steven Honson <<a href="mailto:steven@honson.id.au">steven@honson.id.au</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed, 12 Jun 2019, at 3:56 AM, Jason A. Donenfeld wrote:<br>
> The other thing I was wondering is: aside from performance and races<br>
> as described above, why not just make this the functionality of<br>
> `setconf`? Then there's be no need to introduce a new subcommand. In<br>
> otherwords, the idea would be to make `setconf` not destroy existing<br>
> peers if we're going to be re-adding them again.<br>
<br>
I'm very much in favour of this (updating `setconf` to use this new syncronisation approach), if anything it feels more logical and is how I initially (wrongly) assumed `setconf` behaved when starting out with WireGuard a while back.<br>
_______________________________________________<br>
WireGuard mailing list<br>
<a href="mailto:WireGuard@lists.zx2c4.com" target="_blank">WireGuard@lists.zx2c4.com</a><br>
<a href="https://lists.zx2c4.com/mailman/listinfo/wireguard" rel="noreferrer" target="_blank">https://lists.zx2c4.com/mailman/listinfo/wireguard</a><br>
</blockquote></div>