Skip to content
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

[FIX] setup.nsi: prevent installation of nginx in a user folder #160054

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

lse-odoo
Copy link
Contributor

Before this commit:
If a windows iot odoo installation is made on a user folder (like Desktop), nginx will not be able to start with the following error:
[emerg] 8144#3304: CreateFile() "C:\Users\...\nginx-1.22.0/conf/nginx.conf" failed (5: Access is denied) On which the only solution that we found was to give access to the user folder to the "LocalService" user.

After this commit:
As it is not a feasible solution, we will prevent Odoo's install in a user folder to avoid this issue

opw-3827086

@lse-odoo lse-odoo requested a review from d-fence March 31, 2024 18:47
@robodoo
Copy link
Contributor

robodoo commented Mar 31, 2024

@C3POdoo C3POdoo requested a review from a team March 31, 2024 18:48
@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label Mar 31, 2024
Before this commit:
If a windows iot odoo installation is made on a user folder
(like Desktop), nginx will not be able to start with the following
error:
`[emerg] 8144#3304: CreateFile() "C:\Users\...\nginx-1.22.0/conf/nginx.conf" failed (5: Access is denied)`
On which the only solution that we found was to give access to the
user folder to the "LocalService" user.

After this commit:
As it is not a feasable solution, we will prevent Odoo's install
in a user folder to avoid this issue

opw-3827086
@lse-odoo lse-odoo force-pushed the 16.0-windows_installer_prevent_user_folder_nginx-lse branch from bb5bc7b to f4b57ad Compare April 2, 2024 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OE the report is linked to a support ticket (opw-...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants