Skip to content

Update huly.nginx. Increased file upload size#40

Open
sergeyph wants to merge 1 commit intohcengineering:mainfrom
sergeyph:patch-1
Open

Update huly.nginx. Increased file upload size#40
sergeyph wants to merge 1 commit intohcengineering:mainfrom
sergeyph:patch-1

Conversation

@sergeyph
Copy link
Copy Markdown
Contributor

@sergeyph sergeyph commented Oct 3, 2024

No description provided.

Signed-off-by: sergeyph <168668657+sergeyph@users.noreply.github.com>
Comment thread nginx/huly.nginx
server {
listen 80;
server_name _;
client_max_body_size 100M;
Copy link
Copy Markdown

@egavrilov egavrilov Oct 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
client_max_body_size 100M;
client_max_body_size 0;

Might be even better to lift client_max_body_size and control it through the outer reverse proxy or host configuration, what do you think?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I use nginx as reverse proxy - so there is Huly and nginx in front. Had problems with attaching files to issues in Huly and increased the client_max_body_size

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might be even better to lift client_max_body_size and control it through the outer reverse proxy or host configuration, what do you think?

Agree, if huly could control on own side (not checked yet, just started reading the docs)

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.

3 participants