Merge pull request #143 from betagouv/update/monitor-top-site

Ajoute le support de https sur le site de monitoring
betagouv · Sep 13, 2023 · e3c5257 · e3c5257
2 parents 682bcd9 + b3b09d1
commit e3c5257
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/README.md b/README.md
@@ -54,7 +54,7 @@ This will disable SSH connection to the server using password. This step will no
 - set `PasswordAuthentication no` 
 - set `ChallengeResponseAuthentication no`
 
-It is advised to run this command on a newly installed and if you understand the implication of those parameters. Your hosting service should provide you with an emergency access if you get locked out of the server.
+It is advised to run this command on a newly installed server, while keeping an active ssh connection in parallel and only if you understand the implication of those parameters. Your hosting service should provide you with an emergency access if you get locked out of the server.
 
 ### Listing required DNS record
 

diff --git a/roles/bootstrap/tasks/main.yaml b/roles/bootstrap/tasks/main.yaml
@@ -42,6 +42,7 @@
   vars:
     service_domain: monitor.{{ fullname }}
     upstream_name: monitor
+    https: true
 - name: Provision tasks
   ansible.builtin.include_tasks: nginx_application_sites.yaml
   with_items: "{{ applications }}"