Added Subnet and IP in the Vpn list and Vpn detail APIs #1005
Conversation
Added 'subnet' and 'ip' fields in the /controller/vpn/ and /controller/vpn/${id} get requests
Added the "vpnclient_config" object inside the "config" object when the "controller_device_read" api is called. It's useful to see the ip addresses assigned to a device when connected to a wireguard vpn
|
Hi @nemesifier , I'm using Openwisp API as part of a personal project and I'm happy to contribute (even though I've a very limited availability) Thanks, |
There was a problem hiding this comment.
@marino-mrc thanks for contributing!
I think before the change was quite clear, now it's not very clear what the intended outcome is and I am not sure we can merge it as is.
Keep in mind that if you need to customize the behavior of OpenWISP, changing the core code is not the right approach, you can create a sub app in your django project and extend the API views of OpenWISP, replacing a specific URL with your custom view.
This way your extension becomes more maintainable.
Fixed the problem when a vpn_client template is removed from the configuration of a specific device
Added the possibility to set webhook_endpoint and auth_token when calling a POST on /controller/vpn (the type must be wireguard)
Added 'subnet' and 'ip' fields in the /controller/vpn/ and /controller/vpn/${id} get requests
Checklist
Reference to Existing Issue
Closes #1003
Description of Changes
Changed the Meta calss inside the VpnSerializer class in order to add 'subnet' and 'ip' fields