donaldzou / WGDashboard

Simple dashboard for WireGuard VPN written in Python & Vue.js
https://donaldzou.github.io/WGDashboard-Documentation/
Apache License 2.0
1.53k stars 230 forks source link

API: Bulk Add Peers Fails #425

Open TapanBala opened 2 weeks ago

TapanBala commented 2 weeks ago

Seems the documentation for Bulk Add Peers has incorrect body parameters mentioned.

The described payload throws an exception since the addPeers route is still expecting public_key, allowed_ips and preshared_key in the payload and throws a KeyError exception.

As a temporary workaround passing empty values in the payload works:

"public_key": "",
"allowed_ips": "",
"preshared_key": ""

Also I couldn't find preshared_key_bulkAdd being used anywhere.

donaldzou commented 2 weeks ago

Oh nooo.. I must've pushed the v4.1 documentation to it...my bad

Also I couldn't find preshared_key_bulkAdd being used anywhere.

Yes and this is also only being used in the next release.

vinaynarasimhan commented 2 weeks ago

@donaldzou and @TapanBala , did you guys also notice, untitled files getting created ? when you try to generate files in large number.

i have noticed with 2000, but even have noticed with 50 numbers occasionally .

This is via web portal.

donaldzou commented 1 week ago

@donaldzou and @TapanBala , did you guys also notice, untitled files getting created ? when you try to generate files in large number.

i have noticed with 2000, but even have noticed with 50 numbers occasionally .

This is via web portal.

Hi @vinaynarasimhan , did you choose to use Pre-Shared Key?

TapanBala commented 1 week ago

Oh nooo.. I must've pushed the v4.1 documentation to it...my bad

Ahh no worries. Thanks.