Skip to content

Fix 500 error when not providing Authorization header in Attempts API#6808

Merged
mitchellhenke merged 1 commit intomainfrom
mitchellhenke/attempts-api-500
Aug 19, 2022
Merged

Fix 500 error when not providing Authorization header in Attempts API#6808
mitchellhenke merged 1 commit intomainfrom
mitchellhenke/attempts-api-500

Conversation

@mitchellhenke
Copy link
Contributor

Adds a failing spec and the fix. Current implementation will 500 with:

     NoMethodError:
       undefined method `split' for nil:NilClass

changelog: Bug Fixes, API, Fix 500 error when not providing Authorization header in Attempts API
@mitchellhenke mitchellhenke requested a review from a team August 19, 2022 18:30
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM

@mitchellhenke mitchellhenke merged commit 4f71099 into main Aug 19, 2022
@mitchellhenke mitchellhenke deleted the mitchellhenke/attempts-api-500 branch August 19, 2022 19:05
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.

3 participants