Skip to content

Commit

Permalink
Merge pull request #1391 from pixelpark/feature/master/passenger_user
Browse files Browse the repository at this point in the history
(#3139) Add support for PassengerUser
  • Loading branch information
hunner committed Mar 11, 2016
2 parents 723eb0b + fcc9bc5 commit 8a70b4f
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 2 deletions.
4 changes: 4 additions & 0 deletions README.md
Expand Up @@ -2326,6 +2326,10 @@ Sets [PassengerStartTimeout](https://www.phusionpassenger.com/library/config/apa

Sets [PassengerPreStart](https://www.phusionpassenger.com/library/config/apache/reference/#passengerprestart), the URL of the application if pre-starting is required.

##### `passenger_user`

Sets [PassengerUser](https://www.phusionpassenger.com/library/config/apache/reference/#passengeruser), the running user for sandboxing applications.

##### `php_flags & values`

Allows per-virtual host setting [`php_value`s or `php_flag`s](http://php.net/manual/en/configuration.changes.php). These flags or values can be overwritten by a user or an application. Default: '{}'.
Expand Down
6 changes: 4 additions & 2 deletions manifests/vhost.pp
Expand Up @@ -125,6 +125,7 @@
$passenger_min_instances = undef,
$passenger_start_timeout = undef,
$passenger_pre_start = undef,
$passenger_user = undef,
$add_default_charset = undef,
$modsec_disable_vhost = undef,
$modsec_disable_ids = undef,
Expand Down Expand Up @@ -276,7 +277,7 @@
include ::apache::mod::suexec
}

if $passenger_app_root or $passenger_app_env or $passenger_ruby or $passenger_min_instances or $passenger_start_timeout or $passenger_pre_start {
if $passenger_app_root or $passenger_app_env or $passenger_ruby or $passenger_min_instances or $passenger_start_timeout or $passenger_pre_start or $passenger_user {
include ::apache::mod::passenger
}

Expand Down Expand Up @@ -956,7 +957,8 @@
# - $passenger_min_instances
# - $passenger_start_timeout
# - $passenger_pre_start
if $passenger_app_root or $passenger_app_env or $passenger_ruby or $passenger_min_instances or $passenger_start_timeout or $passenger_pre_start {
# - $passenger_user
if $passenger_app_root or $passenger_app_env or $passenger_ruby or $passenger_min_instances or $passenger_start_timeout or $passenger_pre_start or $passenger_user {
concat::fragment { "${name}-passenger":
target => "${priority_real}${filename}.conf",
order => 300,
Expand Down
1 change: 1 addition & 0 deletions spec/defines/vhost_spec.rb
Expand Up @@ -338,6 +338,7 @@
'passenger_min_instances' => '1',
'passenger_start_timeout' => '600',
'passenger_pre_start' => 'http://localhost/myapp',
'passenger_user' => 'sandbox',
'add_default_charset' => 'UTF-8',
'jk_mounts' => [
{ 'mount' => '/*', 'worker' => 'tcnode1', },
Expand Down
3 changes: 3 additions & 0 deletions templates/vhost/_passenger.erb
Expand Up @@ -16,3 +16,6 @@
<% if @passenger_pre_start -%>
PassengerPreStart <%= @passenger_pre_start %>
<% end -%>
<% if @passenger_user -%>
PassengerUser <%= @passenger_user %>
<% end -%>

0 comments on commit 8a70b4f

Please sign in to comment.