Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expand status codes #138

Merged
merged 2 commits into from Jan 26, 2018
Merged

Expand status codes #138

merged 2 commits into from Jan 26, 2018

Conversation

PENGUINLIONG
Copy link
Contributor

@tomaka
Copy link
Member

tomaka commented Jan 25, 2018

Looks good, thanks!
Coud you please also add an entry to the CHANGELOG?

@PENGUINLIONG
Copy link
Contributor Author

PENGUINLIONG commented Jan 25, 2018

I read the registry of HTTP status code reservation on IANA after that. Maybe it's a better option to use the IANA settings. But how about those WebDAV codes (like 208 Multi-Status, which is 208 Already Reported in IANA registry) already exist in this repo? Should those be kept and populated along with those HTTP codes?

EDIT WebDAV is not a problem actually. WebDAV codes are also registered. But I still wonder why Multi-Status has two different value in MDN's record.

@frewsxcv frewsxcv merged commit ade31f6 into tiny-http:master Jan 26, 2018
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.

None yet

3 participants