-
Notifications
You must be signed in to change notification settings - Fork 62
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
It should be clear in the Readme what protocols it supports #46
Comments
👍 |
Hey guys, we send messages through graylog2 which I think supports UDP only (I might be wrong :-P). In general, though, I would only bother to support UDP in this context. |
@odino graylog2 supports a wide range of inputs and protocols, most prominently GELF over UDP, TCP and HTTP. Having UDP only is OK as a choice (although having HTTP too would be nice) but having the Readme state so would improve the module's UX. |
In case someone needs to support HTTP or TCP, I've written a transport for Winston that supports them: winston-log2gelf. |
@odino Alright, sorry, I hadn't followed your link, believing it would lead me to the Graylog 2 website. |
@kwisatz no worries! I assume that module might let you use other protocols, though Im not sure :) |
@Buzut sure, though even with HTTP you dont get security -- and I believe HTTPS might be too expensive. I would be surprised to see massive deployments with logging happening via HTTPS :) |
@odino With keepalive wisely configured, the overhead isn't that impacting. Plus it depends on what you log and the amount of data. Logging from a website's frontend for instance, is only possibe through HTTP (but that's out of scope for a winston transport anyway). |
Currently it's not very clear what is supported and what is not. From the provided exemple, I can infer that the lib sends data on UDP, but is there a way to support TCP, TCP over TLS, HTTP?
Thank you for making that clear. And thank you for maintaining this project!
The text was updated successfully, but these errors were encountered: