Skip to content
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

Fix bug LIBCLOUD-635 - openstack-swift auth v2 #492

Closed
wants to merge 1 commit into from

Conversation

@fifieldt
Copy link
Contributor

fifieldt commented Apr 1, 2015

As reported by James in LIBCLOUD-635
https://issues.apache.org/jira/browse/LIBCLOUD-635

OpenStack-Swift authentication is broken from at least
0.16. I confirmed the bug in 0.17 and master.

the code here tries to perform operations on a
OpenStackServiceCatalogEntryEndpoint that cannot
be performed on this class.

Since get_endpoint() will return an endpoint with
a URL (not 3 different URLs: publicURL/internalURL/
adminURL) or error, it should be safe to test for
existence and use the url. However, review very
welcome!

This fix has been tested against OpenStack Juno.

As reported by James in LIBCLOUD-635
https://issues.apache.org/jira/browse/LIBCLOUD-635

OpenStack-Swift authentication is broken from at least
 0.16. I confirmed the bug in 0.17 and master.

the code here tries to perform operations on a
OpenStackServiceCatalogEntryEndpoint that cannot
be performed on this class.

Since get_endpoint() will return an endpoint with
a URL (not 3 different URLs: publicURL/internalURL/
adminURL) or error, it should be safe to test for
existence and use the url. However, review very
welcome!

This fix has been tested against OpenStack Juno.
@porterjamesj
Copy link

porterjamesj commented Apr 1, 2015

👍

@Kami
Copy link
Member

Kami commented Apr 1, 2015

Thanks. I will look into it shortly.

@fifieldt
Copy link
Contributor Author

fifieldt commented Apr 3, 2015

Thanks! At the moment at OpenStack upstream we're writing a guide (I think you're going to like it!) which relies on libcloud, including the swift bits, so getting this in soon-ish would be handy :)

@Kami
Copy link
Member

Kami commented Apr 4, 2015

@fifieldt Awesome, great to hear that :)

Do let us know if there is anything we can to do make the experience better / more pleasant. And as always, contributions are welcome :)

@asfgit asfgit closed this in 06cdd4e Apr 4, 2015
@Kami
Copy link
Member

Kami commented Apr 4, 2015

Also, fix merged into trunk. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.