-
-
Notifications
You must be signed in to change notification settings - Fork 130
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for using application credentials for authentication #533
Add support for using application credentials for authentication #533
Conversation
Looks good overall. Unfortunately the test suite has been busted for a bit here, other maintainers haven't really been active, and I don't have an active openstack setup to test against either. Have you been able to test this out successly? |
Thank you
I have tried to add a test, that was succesfull, but I saw there were tests failing, even without my change.
Yes, I'm using the branch with these changes with success and only use application credentials to log in in Openstack. I'm using my branch currently from beaker-openstack where the Compute and the Network class is used directly. I'm not sure if something is missing as I'm not using anything other than those classes. Furthermore, I'm not an expert about the communication with Openstack. This change works for me, but I can't guarantee it works for other cases. |
Thanks for the context, that makes sense to me. The change to nil would be great and then I think we can bring this in. You can only speak for your own experience, but from what I see it shouldn't break anything that is currently working for others either way. |
I have changes |
09ca4a4
to
2a4e5da
Compare
Great, thanks for that. |
@scornelissen85 did you need a release with this change? |
@geemus If you could make a release I can then make a PR for beaker-openstack to use the application credentials. |
…#533) * Add initial support for using application credentials for auth * Add spec test for using application credentials * Add application credentials options to identity * Return nil from scope if application credential is used
* Swift supports application credentials: restic/restic#2156 * see also fog#533
* Swift supports application credentials: restic/restic#2156 * see also #533
I've used this code for using application credentials for authentication in Openstack. I think it would be valuable if this could be added in the fog-openstack project.
This will fix issue #510
This can be used as alternative next to it's original credentials options. To use application credentials I've set the following parameters:
CC @dhague (sorry if I should cc another member)