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

adding support for proxy_cache_bypass and proxy_cache_lock #1150

Merged
merged 2 commits into from
Nov 18, 2017

Conversation

ceonizm
Copy link
Contributor

@ceonizm ceonizm commented Nov 16, 2017

As requested in previous PR (#1145) I've splitted features in multiples branches and PR:

Hello,
As I needed to use proxy_cache_bypass and proxy_cache_lock in my project, I've added it to the module.

Hope it helps.

Best regards
François.

@wyardley
Copy link
Collaborator

Can you add tests for this?

@wyardley wyardley merged commit 6054163 into voxpupuli:master Nov 18, 2017
cegeka-jenkins pushed a commit to cegeka/puppet-nginx that referenced this pull request Sep 13, 2019
…#1150)

* adding support for proxy_cache_bypass and proxy_cache_lock

* adding tests and restrict proxy_cache_lock to enum
ceonizm added a commit to ceonizm/puppet-nginx that referenced this pull request Feb 7, 2020
…#1150)

* adding support for proxy_cache_bypass and proxy_cache_lock

* adding tests and restrict proxy_cache_lock to enum
ceonizm pushed a commit to ceonizm/puppet-nginx that referenced this pull request Feb 7, 2020
adding support for include directive in map (voxpupuli#1149)

* adding support for include directive in map

* change param name includes to include_files to keep naming consistent with the other manifests
adding test to ensure empty value , single and multiple inclusions works correctly

* fix rubocop offenses

adding support for proxy_cache_bypass and proxy_cache_lock (voxpupuli#1150)

* adding support for proxy_cache_bypass and proxy_cache_lock

* adding tests and restrict proxy_cache_lock to enum

	modifié :         manifests/resource/location.pp
	modifié :         manifests/resource/map.pp
	modifié :         manifests/resource/server.pp
	modifié :         spec/defines/resource_location_spec.rb
	modifié :         spec/defines/resource_map_spec.rb
	modifié :         templates/conf.d/map.erb
	modifié :         templates/server/locations/proxy.erb
ceonizm added a commit to ceonizm/puppet-nginx that referenced this pull request Feb 7, 2020
…nd directives

adding documentation comments

adding support for include directive in map (voxpupuli#1149)

* adding support for include directive in map

* change param name includes to include_files to keep naming consistent with the other manifests
adding test to ensure empty value , single and multiple inclusions works correctly

* fix rubocop offenses

adding support for proxy_cache_bypass and proxy_cache_lock (voxpupuli#1150)

* adding support for proxy_cache_bypass and proxy_cache_lock

* adding tests and restrict proxy_cache_lock to enum

fixing access_log, adding error_log and adding tests

access_log: adding format_log to stay consistent with server declaration: when using it just disable logging, error_log: adding error_level to be able to set a different error level per location if needed); passing off disable the error_log

changing type to Optional[Variant[Array, String]] for access_log & error_log to handle multiple declaration

changing type to Optional[Variant[Array, String]] for access_log & error_log to handle multiple declaration
Rubueno pushed a commit to Rubueno/puppet-nginx that referenced this pull request Oct 19, 2020
…#1150)

* adding support for proxy_cache_bypass and proxy_cache_lock

* adding tests and restrict proxy_cache_lock to enum
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