Skip to content

Checking for second unauthorized response after authorized registration fixes #18#25

Open
sischnei wants to merge 1 commit into
RestComm:masterfrom
sischnei:fix-endless-registrations
Open

Checking for second unauthorized response after authorized registration fixes #18#25
sischnei wants to merge 1 commit into
RestComm:masterfrom
sischnei:fix-endless-registrations

Conversation

@sischnei

Copy link
Copy Markdown

Fixes endless registrations when invalid credentials are provided and the server responds with "401 Unauthorized" instead of "403 Forbidden".

@gsaslis

gsaslis commented Jan 30, 2018

Copy link
Copy Markdown
Contributor

Thanks for opening the PR @sischnei !

Could you please take a quick look at our Open Source playbook, and make sure you have signed our Contributors License Agreement (from what I can see you haven't signed it before, have you?)

@sischnei

Copy link
Copy Markdown
Author

@gsaslis I filled out the online form yesterday - anything else missing?

@gsaslis

gsaslis commented Feb 6, 2018

Copy link
Copy Markdown
Contributor

Nope - thanks a lot @sischnei ! (and apologies for the late reply here... :) )

I can now pass this on to @RestComm/vvs-squad for review.

@gsaslis gsaslis requested a review from a team February 6, 2018 09:49
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