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

http status code cannot be outside of range 100-599 #351

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

LorenaSoledad
Copy link
Contributor

Fixes #264

Copy link
Contributor

@svacas svacas left a comment

Choose a reason for hiding this comment

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

once the range is clarified in the spec we can merge this change

@@ -759,6 +759,6 @@ protected Rule protocols()

protected Rule responseCodes()
{
return range(Range.closed(100L, 599L));
return range(Range.closed(0L, 999L));
Copy link
Contributor

Choose a reason for hiding this comment

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

open a spec bug to clarify what should be the allowed range of status codes, otherwise next time somebody will want to use status code 1050 and it will not be possible.

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

Successfully merging this pull request may close these issues.

2 participants