zulip / docker-zulip

Container configurations, images, and examples for Zulip.
https://zulip.com/
Apache License 2.0
550 stars 227 forks source link

/json/messages endpoint returns 500 #427

Open Pandry opened 5 months ago

Pandry commented 5 months ago

Hiya! Thanks for the great work!
I manage a Zulip server for an association and we've been using zulip since version 3 I think.
Since the version 8 update, we had a bit of issues regarding the /json/events endpoint:

image image

Turned out, the issue was the nginx server inside the docker container: 2024/01/22 21:12:43 [error] 56#56: *44354 invalid URL prefix in "localhost:9800", client: 172.18.0.2, server: , request: "DELETE /json/events HTTP/1.1", host: "zulip.*.org", referrer: "https://zulip.*.org/"

As the issue says, the upstream misses the protocol (I assumed http looking around in the same file)
The fix I found was to replace $tornado_server with http://tornado in /etc/nginx/zulip-include/app

Is this an issue unique to our setup? Given the git blame I suspect so, as I tried searching for this issue and possible duplicates and I found no results

If this is indeed an issue, the fix would be as easy as editing the app file, right?