Skip to content
This repository has been archived by the owner on Dec 5, 2023. It is now read-only.

Check status code for explicit 500 #80

Merged
merged 1 commit into from
Mar 16, 2017
Merged

Check status code for explicit 500 #80

merged 1 commit into from
Mar 16, 2017

Conversation

vlal-zz
Copy link
Contributor

@vlal-zz vlal-zz commented Mar 15, 2017

  • Check the body not the response
  • Check for explicit 500 not string

nustiueudinastea
nustiueudinastea approved these changes Mar 16, 2017
View reviewed changes
Copy link
Contributor

@nustiueudinastea nustiueudinastea left a comment

Choose a reason for hiding this comment

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

LGTM

@nustiueudinastea nustiueudinastea merged commit 7adeebc into master Mar 16, 2017
@nustiueudinastea nustiueudinastea deleted the status-code-check branch March 16, 2017 10:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@nustiueudinastea nustiueudinastea nustiueudinastea approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vlal-zz @nustiueudinastea