YunoHost-Apps / piwigo_ynh

Piwigo package for YunoHost
http://piwigo.org/
GNU General Public License v2.0
6 stars 4 forks source link

Force https #29

Closed anmol26s closed 6 years ago

anmol26s commented 6 years ago

Problem

The app is not forcing https, due to which app needs to be manually opened in https to make the communication secure

Solution

Add these line to nginx.conf

if ($scheme = http) {
        rewrite ^ https://$server_name$request_uri? permanent;
}

PR Status

Validation

Micro decision.

Build Status When the PR is marked as ready to merge, you have to wait for 3 days before really merging it.

anmol26s commented 6 years ago

@YunoHost-Apps/apps-group Can you review?

Josue-T commented 6 years ago

Will be merged in 3 days

anmol26s commented 6 years ago

3 days ... :stuck_out_tongue_winking_eye:

anmol26s commented 6 years ago

Can we merge it?