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

✅ Add status tests #15

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

Conversation

AlexV525
Copy link

@AlexV525 AlexV525 commented Dec 2, 2023

Run tests through all HTTP status code.

@AlexV525
Copy link
Author

AlexV525 commented Dec 2, 2023

The service always returns 200 when I run these tests locally...

@AlexV525 AlexV525 marked this pull request as ready for review December 2, 2023 13:27
@AlexV525
Copy link
Author

AlexV525 commented Dec 2, 2023

https://github.com/golang/go/blob/de5b418bea70aaf27de1f47e9b5813940d1e15a4/src/net/http/server.go#L148-L153

	// The provided code must be a valid HTTP 1xx-5xx status code.
	// Any number of 1xx headers may be written, followed by at most
	// one 2xx-5xx header. 1xx headers are sent immediately, but 2xx-5xx
	// headers may be buffered. Use the Flusher interface to send
	// buffered data. The header map is cleared when 2xx-5xx headers are
	// sent, but not with 1xx headers.

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.

1 participant