search

Graylog2 / graylog-docker

Official Graylog Docker image
https://hub.docker.com/r/graylog/graylog/
Apache License 2.0
367 stars 133 forks source link

Check for existing protocol in http_publish_uri #110

Closed structurefall closed 4 years ago

structurefall commented 4 years ago

The Graylog docs indicate that http_publish_uri should include the protocol. This adds a simple check to ensure that we don't specify it twice in check_url.

This is a significant issue in a situation where HTTPS is being handled by a reverse proxy rather than directly by Graylog.

claassistantio commented 4 years ago

CLA assistant check
All committers have signed the CLA.

structurefall commented 4 years ago

Oops, missed the block earlier that should handle this. Gotta go back and figure out why it's not working, sorry.