Remove unused handler, fixed symlink syntax, fix to lms.yml

Fixed the the comments from review. Also noted that the
include in `lms.yml` was in the wrong place (outside the
server block), fixed.

playbooks/roles/lets_encrypt/handlers/main.yml

----
-- name: reload nginx
-  service:
-    name: nginx
-    state: reloaded

playbooks/roles/lets_encrypt/tasks/main.yml

@@ -112,10 +112,10 @@
 # certs they are shared: symlink only required to the first
 
 - name: symlink certificates
-  file: >
-    src=/etc/letsencrypt/live/{{ LETS_ENCRYPT_DOMAINS|first }}/{{ item.src }}
-    dest=/etc/ssl/{{ item.dest }} 
-    state=link
+  file:
+    src: /etc/letsencrypt/live/{{ LETS_ENCRYPT_DOMAINS|first }}/{{ item.src }}
+    dest: /etc/ssl/{{ item.dest }} 
+    state: link
   with_items:
     - { src: 'fullchain.pem', dest: 'certs/fullchain.pem' }
     - { src: 'privkey.pem',  dest: 'private/privkey.pem' }

playbooks/roles/nginx/templates/edx/app/nginx/sites-available/cms.j2

@@ -104,7 +104,7 @@ error_page {{ k }} {{ v }};
     proxy_pass http://cms-backend;
   }
 
-{% include "lets_encrypt.j2" %}
+  {% include "lets_encrypt.j2" %}
 
   location / {
     {% if EDXAPP_CMS_ENABLE_BASIC_AUTH|bool %}

playbooks/roles/nginx/templates/edx/app/nginx/sites-available/lms.j2

@@ -27,8 +27,6 @@ limit_req_zone {{ "$limit_bot_" ~ agent.alias }} zone=agents:10m rate={{ agent.r
 
 {%- endif %}
 
-{% include "lets_encrypt.j2" %}
-
 
 {% if NGINX_EDXAPP_EMBARGO_CIDRS %}
   {%- if NGINX_SET_X_FORWARDED_HEADERS %}
@@ -135,6 +133,8 @@ error_page {{ k }} {{ v }};
     proxy_pass http://lms-backend;
   }
 
+  {% include "lets_encrypt.j2" %}
+
   location / {
     {% if EDXAPP_LMS_ENABLE_BASIC_AUTH|bool %}
       {% include "basic-auth.j2" %}