Skip to content

Commit

Permalink
Dev Removed "Force SSL to 'off'" as it makes no sense . It is either …
Browse files Browse the repository at this point in the history
…'on' now or any other string is interpreted as 'not forced'
  • Loading branch information
c-schmitz committed Nov 19, 2019
1 parent d256bf2 commit b287371
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions application/views/admin/global_settings/_security.php
Expand Up @@ -77,11 +77,10 @@
'name' => 'force_ssl',
'value'=> getGlobalSetting('force_ssl'),
'selectOptions'=>array(
"neither"=>gT("Don't force on/off",'unescaped'),
"on"=>gT("On",'unescaped'),
"off"=>gT("Off",'unescaped')
)
));?>
));?>
</div>
</div>
<?php
Expand Down

1 comment on commit b287371

@Shnoulle
Copy link
Collaborator

Choose a reason for hiding this comment

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

I like to have SSL available or not … according to current url …

Since we can allow https://limesurvey.example.org AND http://limesurvey.example.org at the same time : why not keep it ?

PS : seems some proxy didn't allow Let's Encrypt SSL

Please sign in to comment.