search

fanout / django-eventstream

Server-Sent Events for Django
MIT License
664 stars 88 forks source link

Send_event through multiple instance and process #152

Closed enzofrnt closed 2 months ago

enzofrnt commented 2 months ago

Hi @jkarneges,

Here is my pull request, which includes the Redis version of the multiprocess communication. However, it does not include the Linux socket version, as I was unable to implement it properly as mentioned earlier in our discussion.

Related to #151

enzofrnt commented 2 months ago

Ho @jkarneges I really need this to be resolved this week to release something at work using the new multiprocess version. Please take a look at it as soon as possible. Thank you !

enzofrnt commented 2 months ago

Everything fixed. I push in 1 minute.

enzofrnt commented 2 months ago

Fixed

enzofrnt commented 2 months ago

@jkarneges When I add the self.lock.acquire() in the view.py file, the server does not respond to SSE request. So I delete it.

enzofrnt commented 2 months ago

Can you take a look before the end of the week, I have a release to do, and I would like to release my product with this ^^

jkarneges commented 2 months ago

LGTM. thank you