logspout icon indicating copy to clipboard operation
logspout copied to clipboard

Allow non-browser websocket clients to access the /logs endpoint

Open semk opened this issue 4 years ago • 2 comments

websocket.Handler.ServeHTTP serves to browser requests and was also causing logspout to crash when accessed by a client like wscat as they don't typically set the origin header (See https://github.com/gliderlabs/logspout/issues/412). This change bypasses the validation for origin header.

semk avatar Feb 23 '21 11:02 semk

Thanks for the contribution. Thoughts on a unit test or two for this and the package in general?

michaelshobbs avatar Feb 23 '21 14:02 michaelshobbs

Hei ; the error is sill happening and non-browser client seem to have a problem using the ws endpoint

6r17 avatar Nov 13 '23 14:11 6r17