Skip to content

Commit e94ca9f

Browse files
committed
Fixed: Rewrite proxy for ssl targets
1 parent 1704536 commit e94ca9f

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

server/conf/vhost.conf.master

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -517,6 +517,9 @@
517517
RewriteCond %{REQUEST_URI} !^/php-fcgi/
518518
RewriteCond %{REQUEST_URI} !^<tmpl_var name='rewrite_target'>
519519
</tmpl_if>
520+
<tmpl_if name="rewrite_target_is_ssl" op="==" value="y">
521+
SSLProxyEngine On
522+
</tmpl_if>
520523

521524
RewriteRule ^/(.*)$ <tmpl_var name='rewrite_target'><tmpl_if name="rewrite_add_path" op="==" value="y">$1</tmpl_if> <tmpl_var name='rewrite_type'>
522525

server/plugins-available/apache2_plugin.inc.php

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1372,29 +1372,30 @@ function update($event_name, $data) {
13721372
$rewrite_rules[] = array('rewrite_domain' => '^'.$this->_rewrite_quote($data['new']['domain']),
13731373
'rewrite_type' => ($data['new']['redirect_type'] == 'no')?'':'['.$data['new']['redirect_type'].']',
13741374
'rewrite_target' => $rewrite_target,
1375+
'rewrite_target_is_ssl' => ('https://' === substr($rewrite_target, 0, 8) ? 'y' : 'n'),
13751376
'rewrite_target_ssl' => $rewrite_target_ssl,
13761377
'rewrite_is_url' => ($this->_is_url($rewrite_target) ? 'y' : 'n'),
13771378
'rewrite_add_path' => (substr($rewrite_target, -1) == '/' ? 'y' : 'n'));
13781379
$rewrite_rules[] = array('rewrite_domain' => '^' . $this->_rewrite_quote('www.'.$data['new']['domain']),
13791380
'rewrite_type' => ($data['new']['redirect_type'] == 'no')?'':'['.$data['new']['redirect_type'].']',
13801381
'rewrite_target' => $rewrite_target,
1381-
'rewrite_target_ssl' => $rewrite_target_ssl,
1382+
'rewrite_target_is_ssl' => ('https://' === substr($rewrite_target, 0, 8) ? 'y' : 'n'),
13821383
'rewrite_is_url' => ($this->_is_url($rewrite_target) ? 'y' : 'n'),
13831384
'rewrite_add_path' => (substr($rewrite_target, -1) == '/' ? 'y' : 'n'));
13841385
break;
13851386
case '*':
13861387
$rewrite_wildcard_rules[] = array( 'rewrite_domain' => '(^|\.)'.$this->_rewrite_quote($data['new']['domain']),
13871388
'rewrite_type' => ($data['new']['redirect_type'] == 'no')?'':'['.$data['new']['redirect_type'].']',
13881389
'rewrite_target' => $rewrite_target,
1389-
'rewrite_target_ssl' => $rewrite_target_ssl,
1390+
'rewrite_target_is_ssl' => ('https://' === substr($rewrite_target, 0, 8) ? 'y' : 'n'),
13901391
'rewrite_is_url' => ($this->_is_url($rewrite_target) ? 'y' : 'n'),
13911392
'rewrite_add_path' => (substr($rewrite_target, -1) == '/' ? 'y' : 'n'));
13921393
break;
13931394
default:
13941395
$rewrite_rules[] = array( 'rewrite_domain' => '^'.$this->_rewrite_quote($data['new']['domain']),
13951396
'rewrite_type' => ($data['new']['redirect_type'] == 'no')?'':'['.$data['new']['redirect_type'].']',
13961397
'rewrite_target' => $rewrite_target,
1397-
'rewrite_target_ssl' => $rewrite_target_ssl,
1398+
'rewrite_target_is_ssl' => ('https://' === substr($rewrite_target, 0, 8) ? 'y' : 'n'),
13981399
'rewrite_is_url' => ($this->_is_url($rewrite_target) ? 'y' : 'n'),
13991400
'rewrite_add_path' => (substr($rewrite_target, -1) == '/' ? 'y' : 'n'));
14001401
}

0 commit comments

Comments
 (0)