Skip to content
This repository has been archived by the owner on Jan 15, 2024. It is now read-only.

Adds checking for the word 'unauthorized' to the unauthorized check. #171 #174

Closed
wants to merge 1 commit into from

Conversation

jonhyman
Copy link
Contributor

Looking at the error message I've been having, failed with error "unauthorized" and then looking at errors.rb, #error_message it seems like there was neither a code nor an assertionCode in the response.

As such, I decided to regex the error message for the word unauthorized. A bit shady perhaps, but I'm getting this error frequently and sporadically in production and staging after upgrading to 2.4.1. :-\

@durran
Copy link
Member

durran commented Mar 27, 2013

Yeah I think to support all versions and unexpected encounters this is what we'll have to do until some sort of consistency is provided back from the db with regards to errors.

@durran
Copy link
Member

durran commented Mar 27, 2013

Pulled in, 1.4.5 released.

@durran durran closed this Mar 27, 2013
@jonhyman
Copy link
Contributor Author

Thanks a lot!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants