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

return 409 status code on duplicate email errors #494

Merged

Conversation

ericchiang
Copy link
Contributor

Closes #492

cc @chancez

@chancez
Copy link
Contributor

chancez commented Jun 28, 2016

Whats the expected way to check for this error then? Should I be type asserting for googleapi.Error and then checking the HTTP code, or?

@ericchiang
Copy link
Contributor Author

I think type asserting for googleapi.Error works or just construct your own request using net/http. I'm not super familiar with the googleapi package.

@ericchiang
Copy link
Contributor Author

Going to merge if this looks good to you.

@chancez
Copy link
Contributor

chancez commented Jun 28, 2016

This looks good.

@ericchiang ericchiang merged commit c0eb36c into dexidp:master Jun 28, 2016
@ericchiang ericchiang deleted the return-409-if-resource-is-already-created branch June 28, 2016 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants