Oauth1 not supported

Bug #1524862 reported by Alexander Petrov
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
keystoneauth
Fix Released
Low
Jamie Lennox

Bug Description

Is there a plugin for OAuth? As far I understand that it doesn't exist due to the following:
 I've tried to run the example:

>>> from keystoneauth1.v3.contrib.oauth1 import auth
>>> from keystoneauth1 import session
>>> from keystoneauth1.v3 import client
>>> a = auth.OAuth('http://my.keystone.com:5000/v3',
... consumer_key=consumer_id,
... consumer_secret=consumer_secret,
... access_key=access_token_key,
... access_secret=access_token_secret)
>>> s = session.Session(auth=auth)

and got the error:

 from keystoneauth1.v3.contrib.oauth1 import auth
---------------------------------------------------------------------------
ImportError Traceback (most recent call last)
<ipython-input-1-5ee1cacfaf35> in <module>()
----> 1 from keystoneauth1.v3.contrib.oauth1 import auth

ImportError: No module named v3.contrib.oauth1

Perhaps it is necessary to improve the documentation or exclude this part.

Revision history for this message
Morgan Fainberg (mdrnstm) wrote :

Improving the documentation and/or removing this is a fine approach.

Changed in keystoneauth:
status: New → Triaged
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystoneauth (master)

Fix proposed to branch: master
Review: https://review.openstack.org/304247

Changed in keystoneauth:
assignee: nobody → Alexander Makarov (amakarov)
status: Triaged → In Progress
Revision history for this message
Jamie Lennox (jamielennox) wrote : Re: Documentation: V3 OAuth 1.0a Plugins

Yea, so these paths look like the keystoneclient paths, not the keystoneauth paths:

>>> from keystoneauth1.v3.contrib.oauth1 import auth
>>> from keystoneauth1 import session
>>> from keystoneauth1.v3 import client

that's not how things are structured in keystoneauth. So i'm guessing the documentation was hastily updated from keystoneclient to keystoneauth and this was missed. It should be a doc fix at least, but we should also bring the oauth plugins over because i don't see them in ksa.

summary: - Documentation: V3 OAuth 1.0a Plugins
+ Oauth1 not supported
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystoneauth (master)

Fix proposed to branch: master
Review: https://review.openstack.org/314401

Changed in keystoneauth:
assignee: Alexander Makarov (amakarov) → Jamie Lennox (jamielennox)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystoneauth (master)

Reviewed: https://review.openstack.org/314401
Committed: https://git.openstack.org/cgit/openstack/keystoneauth/commit/?id=c10722b7894d27d6314c41326f096fc6f81b4c0f
Submitter: Jenkins
Branch: master

commit c10722b7894d27d6314c41326f096fc6f81b4c0f
Author: Jamie Lennox <email address hidden>
Date: Mon May 9 17:32:45 2016 +1000

    Add oauth plugin to keystoneauth

    OAuth1 has been supported by keystone for a long time, and was supported
    as an authentication plugin in keystoneclient. Port this work to
    keystoneauth and add the ability to load it from the CLI.

    Closes-Bug: #1524862
    Change-Id: Ie53aeb1b926104cac692cd98551a701522f7fec4

Changed in keystoneauth:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on keystoneauth (master)

Change abandoned by Steve Martinelli (<email address hidden>) on branch: master
Review: https://review.openstack.org/304247
Reason: https://review.openstack.org/#/c/314401/ merged

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/keystoneauth 2.7.0

This issue was fixed in the openstack/keystoneauth 2.7.0 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.