search

n8n-io / n8n

Free and source-available fair-code licensed workflow automation tool. Easily automate tasks across different services.
https://n8n.io
Other
48.54k stars 7.61k forks source link

SSE node is not working. #1692

Closed santiblanko closed 3 years ago

santiblanko commented 3 years ago

Describe the bug this node is not working all the past year. Also the node doesn't have headers configuration for auth.

To Reproduce Steps to reproduce the behavior:

  1. Go to nodes
  2. Click on SSE trigger
  3. Add a sse endpoint
  4. play and try to receive events.

Expected behavior you need to receive events from the sse server.

Environment (please complete the following information):

janober commented 3 years ago

I just tested it and it works totally fine for me. We did also not change the node since it has been created, so can also not think of a reason why it should have stopped working.

Are you sure that it is connecting correctly and that you then send an SSE event in the time that n8n is listening (so after you pressing "Execute Workflow" and then 120 seconds)?

santiblanko commented 3 years ago

Check:

n8n server for testing: https://hello-world-santiblanko.cloud.okteto.net/workflow/1 SSE server for testing: https://sse-growup.herokuapp.com/sse POST SSE server push https://sse-growup.herokuapp.com/push { "input": "123", "eventName": "example" }

No messages are received. Also a good idea is an option for change the headers for example for add authorization.

janober commented 3 years ago

None of the above URLs does seem to work for me. Is the server running?

Yes agree, is for sure a good idea to add authorization. Sadly are there literally a million things we could add but simply not enough developers to do so. So we have to prioritize. If you are interested in that feature please post a Feature-Request here: https://community.n8n.io/ Other people can then vote and we so prioritize accordingly. Thanks!

ivov commented 3 years ago

@santiblanko The above URLs do not seem to be working. Are you still facing this issue?

ivov commented 3 years ago

Closing for now. If the issue persists, feel free to comment here and we will reopen.

lazmo88 commented 1 year ago

None of the above URLs does seem to work for me. Is the server running?

Yes agree, is for sure a good idea to add authorization. Sadly are there literally a million things we could add but simply not enough developers to do so. So we have to prioritize. If you are interested in that feature please post a Feature-Request here: https://community.n8n.io/ Other people can then vote and we so prioritize accordingly. Thanks!

It seems SSE authentication has not yet been implemented, would it be possible to add this as feature requests?