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

Wsgi hash documentation #2553

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

virot
Copy link

@virot virot commented Jun 10, 2024

Summary

The code says wsgi_daemon_process_options is depreciated

Has been since 0d5e0be

The documentation for that was reverted with 98cf15c

@virot virot requested review from bastelfreak, ekohl, smortex and a team as code owners June 10, 2024 15:01
@CLAassistant
Copy link

CLAassistant commented Jun 10, 2024

CLA assistant check
All committers have signed the CLA.

manifests/vhost.pp Outdated Show resolved Hide resolved
Comment on lines 1193 to 1194
# processes => '2',
# threads => '15',
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do you know if we can use integers? If so, this may be preferable for consistency (even if there is no actual check of the data type).

virot and others added 3 commits June 17, 2024 13:26
Co-authored-by: Romain Tartière <[email protected]>
Template does not care:
WSGIDaemonProcess <%= $key %> <%= join(sort($val.map |$k,$v| { "$k=$v"}), ' ') %>
@virot
Copy link
Author

virot commented Jun 17, 2024

I updated the documentation to Integers instead of Strings.

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.

using wsgi_daemon_process_options logs a deprecation warning but the docs only show the deprecated usage
3 participants