-
Notifications
You must be signed in to change notification settings - Fork 165
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
Enable/disable HTTP ingest #1446
Conversation
server/mediaserver.go
Outdated
@@ -578,6 +585,11 @@ func getRTMPStreamHandler(s *LivepeerServer) func(url *url.URL) (stream.RTMPVide | |||
|
|||
// HandlePush processes request for HTTP ingest | |||
func (s *LivepeerServer) HandlePush(w http.ResponseWriter, r *http.Request) { | |||
if !s.HTTPIngest { | |||
respondWith400(w, "HTTP ingest is disabled") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Arguably an HTTP 405 "Method Not Allowed" but idk if we observe that sort of thing at all in go-livepeer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in 727d98b
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚢
f51a50b
to
1e079cf
Compare
What does this pull request do? Explain your changes. (required)
This PR enables/disables HTTP ingest according to the following rules:
-httpAddr
uses a non-local host) and there is no authentication webhook URL specified then HTTP ingest is disabled-authWebhookUrl
-httpIngest
Specific updates (required)
See commit history for a description of each update.
How did you test each of these updates (required)
Updated unit tests and tested manually.
Does this pull request close any open issues?
Fixes #1441
Checklist:
./test.sh
pass