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

Corrective fixes for new dockerfile #663

Merged
merged 1 commit into from
Nov 29, 2024
Merged

Conversation

turbo124
Copy link
Member

Fixes for nginx response buffers, and remove if checks around architecture that prevented mysql client from installing.

@benbrummer
Copy link
Collaborator

Response buffers are now handled twice

fastcgi_buffer_size 16k;

and
fastcgi_buffer_size 16k;

Cuase of the alphabetical order laravel.conf will be the effective ones

@turbo124
Copy link
Member Author

I don't think those were loading in the correct block. The error message persisted until I added the reference to the fastcgi block in the vhost itself

@benbrummer
Copy link
Collaborator

benbrummer commented Nov 29, 2024

@turbo124 for me the fastcgi_buffer_size is still to small. Same behavior with setting it at either location. But it happens only with the desktop app.

The directives from invoiceninja.conf are loaded in http {...} context
Following the configuration inheritance global > http > server > location it will be set already for the location

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants