Skip to content

feat(auth): add auth:whoami endpoint#36

Merged
bacongobbler merged 1 commit intodeis:masterfrom
bacongobbler:auth-whoami
Jul 25, 2016
Merged

feat(auth): add auth:whoami endpoint#36
bacongobbler merged 1 commit intodeis:masterfrom
bacongobbler:auth-whoami

Conversation

@bacongobbler
Copy link
Copy Markdown
Member

Comment thread deis.go Outdated
//
// If the SDK detects an API version mismatch, it will return ErrAPIMismatch.
const APIVersion = "2.1"
const APIVersion = "2.2"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does this patch need to be delayed until the 2.2 release? also, we can toggle this at build time

Copy link
Copy Markdown
Member Author

@bacongobbler bacongobbler Jul 11, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nah this is the API version bump rather than the platform version bump. These are versioned separately, but because we've had new features in the API these past two releases they are identical. We only bump the API once between platform releases, if necessary. :)

Note that deis/controller#737 also bumps the API version, and deis/workflow#367 adds a v2.2 API reference doc.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

However this is stuck until all the PRs go green though so we don't have an API version mismatch between the client and the server.

I can PR this separately if you'd like!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah, right. I forget about the distinction. thanks for reminding!

@arschles
Copy link
Copy Markdown
Member

LGTM2 assuming #36 (comment) is addressed

@arschles arschles added the LGTM2 label Jul 11, 2016
@bacongobbler bacongobbler removed this from the v2.2 milestone Jul 14, 2016
@bacongobbler bacongobbler added this to the v2.3 milestone Jul 21, 2016
@bacongobbler bacongobbler merged commit cd8ecc1 into deis:master Jul 25, 2016
@bacongobbler bacongobbler deleted the auth-whoami branch July 25, 2016 18:56
cscatolini pushed a commit to topfreegames/controller-sdk-go that referenced this pull request Mar 15, 2018
feat(auth): add auth:whoami endpoint
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants