Skip to content
This repository has been archived by the owner on Mar 3, 2022. It is now read-only.

Updated HTTP Status Codes #51

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

mikestowe
Copy link

Replaced recommendation to use three ambiguous status codes with actual standard status codes as ambiguous status codes or categories can be obtained via the first # in the status code sequence (ie 2xx, 3xx, 4xx, 5xx) and prevents the client from understanding what action is actually taking place (was an item updated or created with a PUT, has the delete been performed, is the item missing or has it been removed, is it a malformed request or do I not have permission to do it, etc).

Replaced recommendation to use three ambiguous status codes with actual standard status codes as ambiguous status codes or categories can be obtained via the first # in the status code sequence (ie 2xx, 3xx, 4xx, 5xx) and prevents the client from understanding what action is actually taking place (was an item updated or created with a PUT, has the delete been performed, is the item missing or has it been removed, is it a malformed request or do I not have permission to do it, etc).
Knew I forgot something :-p - Added the 5XX back into the status codes - universal for "we have no idea what actually happened."
@mikestowe
Copy link
Author

#12

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.

1 participant