cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kansal <...@git.apache.org>
Subject [GitHub] cloudstack pull request: CLOUDSTACK-9162: Unable to add VPN user v...
Date Mon, 14 Dec 2015 07:23:19 GMT
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 <kansal.k@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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message