documentation icon indicating copy to clipboard operation
documentation copied to clipboard

[16.0] install doc for nginx: add map $http_upgrade $connection_upgrade

Open jcdrubay opened this issue 3 years ago • 2 comments

I think you should add this part to the nginx sample configuration as the current one is not working without this:

    map $http_upgrade $connection_upgrade {
        default upgrade;
        '' close;
    }

https://www.nginx.com/blog/websocket-nginx/

jcdrubay avatar Sep 22 '22 01:09 jcdrubay

Hi @jcdrubay , thank you for your feedback.

Hello @d-fence Unless I'm mistaken, I think this is for you :)

jcs-odoo avatar Sep 22 '22 08:09 jcs-odoo

FYI @Julien00859

AntoineVDV avatar Sep 22 '22 09:09 AntoineVDV

I disagree. My Odoo 16 install works without that. AFAIK WebSocket is only required for the chat box.

zeroheure avatar Oct 23 '22 21:10 zeroheure

Thanks for the follow-up @zeroheure

Isn't the chatbox installed/available in most of odoo installations with the module mail?

I tried without that nginx configuration and it seems to work well now. So maybe it was related to a defect that was fixed.

Maybe this configuration is not required anymore after https://github.com/odoo/odoo/commit/a26c8aa18ad6

I will monitor more in coming days.

jcdrubay avatar Oct 24 '22 02:10 jcdrubay

Isn't the chatbox installed/available in most of odoo installations with the module mail?

Sorry, I was talking about the website chatbox, which use WebSocket now, instead of longpolling in previous versions.

zeroheure avatar Oct 24 '22 09:10 zeroheure

The issue was raised internally by other developers, I'm +1 we add it to the example nginx snippet

Julien00859 avatar Oct 27 '22 08:10 Julien00859

#2807

Julien00859 avatar Oct 27 '22 08:10 Julien00859