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

Add whoami API #3423

Merged
merged 3 commits into from Jan 16, 2017

Conversation

Projects
None yet
3 participants
@erasche
Copy link
Member

commented Jan 13, 2017

Simple API route to expose who the current user is.

def whoami( self, trans, **kwd ):
"""
def whoami( self, trans, **kwd )
* GET /api/authenticate/whoami

This comment has been minimized.

Copy link
@nsoranzo

nsoranzo Jan 13, 2017

Member

Should this be just * GET /api/whoami ?

This comment has been minimized.

Copy link
@erasche

erasche Jan 13, 2017

Author Member

Just copied from another set of API docs. Will rewrite, thanks!

@expose_api_anonymous
def whoami( self, trans, **kwd ):
"""
def whoami( self, trans, **kwd )

This comment has been minimized.

Copy link
@nsoranzo

nsoranzo Jan 13, 2017

Member

I don't think this comment line is needed.

* GET /api/authenticate/whoami
returns information about the current authenticated user
:returns: user information in json format

This comment has been minimized.

Copy link
@nsoranzo

nsoranzo Jan 13, 2017

Member

dictionary with user information ?

This comment has been minimized.

Copy link
@erasche

erasche Jan 13, 2017

Author Member

+1

@expose_api_anonymous
def whoami( self, trans, **kwd ):
"""
GET /api/authenticate/whoami

This comment has been minimized.

Copy link
@nsoranzo

nsoranzo Jan 13, 2017

Member

Still need to remove authenticate/ here.

This comment has been minimized.

Copy link
@erasche

erasche Jan 13, 2017

Author Member

Oh! How did I miss that. Cheers.

This comment has been minimized.

Copy link
@erasche

erasche Jan 13, 2017

Author Member

Fixed. I feel a bit like we shouldn't include that and instead have it auto-constructed in the docs?

This comment has been minimized.

Copy link
@nsoranzo

nsoranzo Jan 13, 2017

Member

Thanks! That would be good, but not sure how easy it is.

@martenson martenson modified the milestones: 17.01, 17.05 Jan 16, 2017

@martenson martenson merged commit dd2ab48 into galaxyproject:dev Jan 16, 2017

4 checks passed

api test Build finished. 256 tests run, 0 skipped, 0 failed.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
framework test Build finished. 134 tests run, 0 skipped, 0 failed.
Details
toolshed test Build finished. 580 tests run, 0 skipped, 0 failed.
Details

@erasche erasche deleted the erasche:whoami branch Aug 21, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.