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.

Sign up for GitHub

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

Jump to bottom

[FEATURE] Template proxy buffer vars in nginx.conf #114

Closed
frantsao opened this issue Oct 26, 2022 · 0 comments
Closed

[FEATURE] Template proxy buffer vars in nginx.conf #114

frantsao opened this issue Oct 26, 2022 · 0 comments
Assignees
@frantsao
Labels
enhancement

Comments

@frantsao
Copy link
Contributor

Description

In order to improve the changes added because #62 we need to put in vars the parameters:
proxy buffering
proxy_buffers
proxy_buffer_size
proxy_busy_buffers_size

Why is this needed?

We could override the entire nginx.conf template in playbooks, but these parameters are changed often.

Additional Information

We will try to do not a breaking change (i.e. if there are no changes in playbook, ansible will put the same default values).
@frantsao frantsao self-assigned this Oct 26, 2022
frantsao pushed a commit that referenced this issue Oct 26, 2022
#114 added proxy buffer vars in nginx.conf template
ef3cda9
frantsao pushed a commit that referenced this issue Oct 26, 2022
#114 updated changelog
ab85dae
@frantsao frantsao mentioned this issue Oct 26, 2022
Merged
frantsao pushed a commit that referenced this issue Oct 27, 2022
#114 fix travis tests
72631b5
frantsao pushed a commit that referenced this issue Oct 27, 2022
#114 fix travis tests (2)
bbd3093
frantsao pushed a commit that referenced this issue Oct 27, 2022
#114 BREAKING revert proxy buffering defaults to <2.0 release. They a…
095ae69 
…re safer in general, but suboptimal (see Nginx documentation)
frantsao pushed a commit that referenced this issue Oct 27, 2022
#114 fix files and directories permissions; RENAMED luaJIT_ variables…
3e671de 
… to luajit_ because naming compliance (BREAKING)
frantsao pushed a commit that referenced this issue Oct 27, 2022
#114 updated changelog
a8bc7f4
frantsao pushed a commit that referenced this issue Nov 2, 2022
#114 added role namespace to luajit vars
45728c7
frantsao pushed a commit that referenced this issue Nov 2, 2022
#114 updated changelog and readme
b00d43c
@frantsao frantsao mentioned this issue Nov 2, 2022
Merged
frantsao pushed a commit that referenced this issue Nov 2, 2022
#114 fix extra checks that are not longer needed in templates
da7c80b
@frantsao frantsao closed this as completed Nov 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@frantsao frantsao

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@frantsao