GitHub user kansal opened a pull request:
https://github.com/apache/cloudstack/pull/1241
CLOUDSTACK-9162: Unable to add VPN user via API with Required Parametâ¦
â¦ers - Fixed
RCA:
When we do not provide account parameter which is an optional field , API
picks up the account from which we are logged in. So this works fine with
cloudmonkey and ACS UI. But if we fire API directly from the browser(Make sure
no cookies are stored and you are not logged in), by default system account is
picked and the VPN users addition fails. This behaviour is expected, but no
exception is thrown. So users think that VPN user is created.
Fix: Added an exception when VPN users are added in the system account.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/kansal/cloudstack CLOUDSTACK-9162
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/1241.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1241
----
commit b8497b2a375b256a9f96ffaaaebd0829c88fd45c
Author: Kshitij Kansal <kansa...@gmail.com>
Date: 2015-12-14T07:16:40Z
CLOUDSTACK-9162: Unable to add VPN user via API with Required Parameters -
Fixed
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
