From 6a3d7251347ca2dc5e9436998afb6747841c550c Mon Sep 17 00:00:00 2001
From: Paul Chavard <git@paul.chavard.net>
Date: Wed, 30 Oct 2019 12:11:45 +0100
Subject: [PATCH 01/11] Revert "Revert "Decommission ActiveStorage proxy
 service and use openstack service""

This reverts commit 71227be37f237fbf2ef374a623b6913d88b86d29.
---
 .../service/ds_proxy_service.rb               | 57 -------------------
 config/environments/production.rb             |  2 +-
 config/initializers/active_storage.rb         | 30 ++++++++++
 config/storage.yml                            |  3 -
 4 files changed, 31 insertions(+), 61 deletions(-)
 delete mode 100644 app/lib/active_storage/service/ds_proxy_service.rb

diff --git a/app/lib/active_storage/service/ds_proxy_service.rb b/app/lib/active_storage/service/ds_proxy_service.rb
deleted file mode 100644
index fe540d591..000000000
--- a/app/lib/active_storage/service/ds_proxy_service.rb
+++ /dev/null
@@ -1,57 +0,0 @@
-module ActiveStorage
-  # Wraps an ActiveStorage::Service to route direct upload and direct download URLs through our proxy,
-  # thus avoiding exposing the storage provider’s URL to our end-users.
-  class Service::DsProxyService < SimpleDelegator
-    attr_reader :wrapped
-
-    def self.build(wrapped:, configurator:, **options)
-      new(wrapped: configurator.build(wrapped))
-    end
-
-    def initialize(wrapped:)
-      @wrapped = wrapped
-      super(wrapped)
-    end
-
-    def url(*args)
-      url = wrapped.url(*args)
-      publicize(url)
-    end
-
-    def url_for_direct_upload(*args)
-      url = wrapped.url_for_direct_upload(*args)
-      publicize(url)
-    end
-
-    private
-
-    def object_for(key, &block)
-      blob_url = url(key)
-      if block_given?
-        request = Typhoeus::Request.new(blob_url)
-        request.on_headers do |response|
-          if response.code != 200
-            raise Fog::OpenStack::Storage::NotFound.new
-          end
-        end
-        request.on_body do |chunk|
-          yield chunk
-        end
-        request.run
-      else
-        response = Typhoeus.get(blob_url)
-        if response.success?
-          response
-        else
-          raise Fog::OpenStack::Storage::NotFound.new
-        end
-      end
-    end
-
-    def publicize(url)
-      search = %r{^https://[^/]+/v1/AUTH_[a-f0-9]{32}}
-      replace = 'https://static.demarches-simplifiees.fr'
-      url.gsub(search, replace)
-    end
-  end
-end
diff --git a/config/environments/production.rb b/config/environments/production.rb
index 0c93feb51..8837fb88b 100644
--- a/config/environments/production.rb
+++ b/config/environments/production.rb
@@ -93,7 +93,7 @@ Rails.application.configure do
   # the I18n.default_locale when a translation cannot be found).
   config.i18n.fallbacks = true
 
-  config.active_storage.service = :proxied
+  config.active_storage.service = :openstack
 
   # Send deprecation notices to registered listeners.
   config.active_support.deprecation = :notify
diff --git a/config/initializers/active_storage.rb b/config/initializers/active_storage.rb
index 7357c9267..1d600d89c 100644
--- a/config/initializers/active_storage.rb
+++ b/config/initializers/active_storage.rb
@@ -7,3 +7,33 @@ ActiveStorage::Service.url_expires_in = 1.hour
 # cleaner (as it allows to enqueue the virus scan on attachment creation, rather
 # than on blob creation).
 ActiveSupport.on_load(:active_storage_blob) { include BlobVirusScanner }
+
+# When an OpenStack service is initialized it makes a request to fetch
+# `publicURL` to use for all operations. We intercept the method that reads
+# this url and replace the host with DS_Proxy host. This way all the operation
+# are performed through DS_Proxy.
+#
+# https://github.com/fog/fog-openstack/blob/37621bb1d5ca78d037b3c56bd307f93bba022ae1/lib/fog/openstack/auth/catalog/v2.rb#L16
+require 'fog/openstack/auth/catalog/v2'
+
+module Fog::OpenStack::Auth::Catalog
+  class V2
+    def endpoint_url(endpoint, interface)
+      url = endpoint["#{interface}URL"]
+
+      if interface == 'public'
+        publicize(url)
+      else
+        url
+      end
+    end
+
+    private
+
+    def publicize(url)
+      search = %r{^https://[^/]+/}
+      replace = 'https://static.demarches-simplifiees.fr/'
+      url.gsub(search, replace)
+    end
+  end
+end
diff --git a/config/storage.yml b/config/storage.yml
index 11de850f6..0427a3f7a 100644
--- a/config/storage.yml
+++ b/config/storage.yml
@@ -4,9 +4,6 @@ local:
 test:
   service: Disk
   root: <%= Rails.root.join("tmp/storage") %>
-proxied:
-  service: DsProxy
-  wrapped: openstack
 openstack:
   service: OpenStack
   container: "<%= ENV['FOG_ACTIVESTORAGE_DIRECTORY'] %>"

From 5a87db9920e2d24fd110623c29d1b20efd1f3c0d Mon Sep 17 00:00:00 2001
From: Paul Chavard <git@paul.chavard.net>
Date: Wed, 30 Oct 2019 12:11:53 +0100
Subject: [PATCH 02/11] Revert "Revert "Update activestorage-openstack""

This reverts commit c102dc63b77b1b9c1a6a080bd786709b180b887a.
---
 Gemfile      |  2 +-
 Gemfile.lock | 18 ++++++------------
 2 files changed, 7 insertions(+), 13 deletions(-)

diff --git a/Gemfile b/Gemfile
index 77978a26f..13763e699 100644
--- a/Gemfile
+++ b/Gemfile
@@ -4,7 +4,7 @@ gem 'aasm'
 gem 'actiontext', git: 'https://github.com/kobaltz/actiontext.git', branch: 'archive', require: 'action_text' # Port of ActionText to Rails 5
 gem 'active_link_to' # Automatically set a class on active links
 gem 'active_model_serializers'
-gem 'activestorage-openstack', git: 'https://github.com/fredZen/activestorage-openstack.git', branch: 'frederic/fix_upload_signature'
+gem 'activestorage-openstack'
 gem 'administrate'
 gem 'after_party'
 gem 'anchored'
diff --git a/Gemfile.lock b/Gemfile.lock
index 6ac80df50..bbfe3c9ab 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -1,14 +1,3 @@
-GIT
-  remote: https://github.com/fredZen/activestorage-openstack.git
-  revision: c71d5107a51701eab9d9267dd0000e6c1cf3e39a
-  branch: frederic/fix_upload_signature
-  specs:
-    activestorage-openstack (0.5.0)
-      fog-openstack (~> 1.0)
-      marcel
-      mime-types
-      rails (~> 5.2.0)
-
 GIT
   remote: https://github.com/kobaltz/actiontext.git
   revision: ef59c4ba99d1b7614dd47f5a294eef553224db88
@@ -75,6 +64,11 @@ GEM
       actionpack (= 5.2.2.1)
       activerecord (= 5.2.2.1)
       marcel (~> 0.3.1)
+    activestorage-openstack (1.0.0)
+      fog-openstack (~> 1.0)
+      marcel
+      mime-types
+      rails (<= 6)
     activesupport (5.2.2.1)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 0.7, < 2)
@@ -717,7 +711,7 @@ DEPENDENCIES
   actiontext!
   active_link_to
   active_model_serializers
-  activestorage-openstack!
+  activestorage-openstack
   administrate
   after_party
   anchored

From de8a1637072f102127c0170d83cffb26f784ebaf Mon Sep 17 00:00:00 2001
From: Paul Chavard <git@paul.chavard.net>
Date: Wed, 30 Oct 2019 12:12:31 +0100
Subject: [PATCH 03/11] Bump openstack and fog-openstack

---
 Gemfile.lock | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/Gemfile.lock b/Gemfile.lock
index bbfe3c9ab..7eaa820eb 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -208,7 +208,7 @@ GEM
     ethon (0.11.0)
       ffi (>= 1.3.0)
     eventmachine (1.2.7)
-    excon (0.62.0)
+    excon (0.68.0)
     execjs (2.7.0)
     factory_bot (4.11.1)
       activesupport (>= 3.0.0)
@@ -232,7 +232,7 @@ GEM
     fog-json (1.2.0)
       fog-core
       multi_json (~> 1.10)
-    fog-openstack (1.0.6)
+    fog-openstack (1.0.10)
       fog-core (~> 2.1)
       fog-json (>= 1.0)
       ipaddress (>= 0.8)
@@ -319,7 +319,7 @@ GEM
       rails-dom-testing (>= 1, < 3)
       railties (>= 4.2.0)
       thor (>= 0.14, < 2.0)
-    json (2.1.0)
+    json (2.2.0)
     json-jwt (1.10.0)
       activesupport (>= 4.2)
       aes_key_wrap
@@ -369,16 +369,16 @@ GEM
     marcel (0.3.3)
       mimemagic (~> 0.3.2)
     method_source (0.9.2)
-    mime-types (3.2.2)
+    mime-types (3.3)
       mime-types-data (~> 3.2015)
-    mime-types-data (3.2018.0812)
+    mime-types-data (3.2019.1009)
     mimemagic (0.3.3)
     mini_mime (1.0.2)
     mini_portile2 (2.4.0)
     minitest (5.11.3)
     momentjs-rails (2.20.1)
       railties (>= 3.1)
-    multi_json (1.13.1)
+    multi_json (1.14.1)
     multi_xml (0.6.0)
     multipart-post (2.0.0)
     mustermann (1.0.3)
@@ -419,7 +419,7 @@ GEM
       validate_email
       validate_url
       webfinger (>= 1.0.1)
-    openstack (3.3.20)
+    openstack (3.3.21)
       json
     orm_adapter (0.5.0)
     parallel (1.12.1)

From def9c67c564ecc32d4e18d82f2236e6578db3d46 Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Thu, 24 Oct 2019 22:09:54 +0200
Subject: [PATCH 04/11] Clean html

---
 .../new_administrateur/groupe_instructeurs/show.html.haml     | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/app/views/new_administrateur/groupe_instructeurs/show.html.haml b/app/views/new_administrateur/groupe_instructeurs/show.html.haml
index 40a3df3f4..393726bd0 100644
--- a/app/views/new_administrateur/groupe_instructeurs/show.html.haml
+++ b/app/views/new_administrateur/groupe_instructeurs/show.html.haml
@@ -5,9 +5,7 @@
                     @groupe_instructeur.label] }
 
 .container.groupe-instructeur
-  .rename_form_block
-    .flex.baseline-start
-      %h1 Groupe « #{@groupe_instructeur.label} »
+  %h1 Groupe « #{@groupe_instructeur.label} »
 
   .card.mt-2
     = form_for @groupe_instructeur,

From 07aefdd40b9f7417fc9f0f04ea282dae198941c0 Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Fri, 25 Oct 2019 10:19:58 +0200
Subject: [PATCH 05/11] Fix test in new_admin/groupe_instructeur_controller

---
 .../groupe_instructeurs_controller_spec.rb             | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb b/spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb
index 6b0d388a4..d13e3d8b4 100644
--- a/spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb
+++ b/spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb
@@ -118,17 +118,17 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
 
     before { gi_1_1.instructeurs << admin.instructeur << instructeur }
 
-    def remove_instructeur(email)
+    def remove_instructeur(instructeur)
       delete :remove_instructeur,
         params: {
           procedure_id: procedure.id,
           id: gi_1_1.id,
-          instructeur: { id: admin.instructeur.id }
+          instructeur: { id: instructeur.id }
         }
     end
 
     context 'when there are many instructeurs' do
-      before { remove_instructeur(admin.user.email) }
+      before { remove_instructeur(admin.instructeur) }
 
       it { expect(gi_1_1.instructeurs).to include(instructeur) }
       it { expect(gi_1_1.reload.instructeurs.count).to eq(1) }
@@ -137,8 +137,8 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
 
     context 'when there is only one instructeur' do
       before do
-        remove_instructeur(admin.user.email)
-        remove_instructeur(instructeur.email)
+        remove_instructeur(admin.instructeur)
+        remove_instructeur(instructeur)
       end
 
       it { expect(gi_1_1.instructeurs).to include(instructeur) }

From 4491dca19aea0a34f8af14e20407b861b60a1846 Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Thu, 24 Oct 2019 21:52:05 +0200
Subject: [PATCH 06/11] Index Groupe Instructeur

---
 .../groupe_instructeurs_controller.rb         | 28 +++++++++++++++++
 .../groupe_instructeurs/index.html.haml       | 19 ++++++++++++
 config/routes.rb                              |  2 ++
 .../groupe_instructeurs_controller_spec.rb    | 31 +++++++++++++++++++
 4 files changed, 80 insertions(+)
 create mode 100644 app/controllers/instructeurs/groupe_instructeurs_controller.rb
 create mode 100644 app/views/instructeurs/groupe_instructeurs/index.html.haml
 create mode 100644 spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb

diff --git a/app/controllers/instructeurs/groupe_instructeurs_controller.rb b/app/controllers/instructeurs/groupe_instructeurs_controller.rb
new file mode 100644
index 000000000..bae72aa36
--- /dev/null
+++ b/app/controllers/instructeurs/groupe_instructeurs_controller.rb
@@ -0,0 +1,28 @@
+module Instructeurs
+  class GroupeInstructeursController < InstructeurController
+    ITEMS_PER_PAGE = 25
+
+    def index
+      @procedure = procedure
+      @groupes_instructeurs = paginated_groupe_instructeurs
+    end
+
+    private
+
+    def procedure
+      current_instructeur
+        .procedures
+        .includes(:groupe_instructeurs)
+        .find(params[:procedure_id])
+    end
+
+    def paginated_groupe_instructeurs
+      current_instructeur
+        .groupe_instructeurs
+        .where(procedure: procedure)
+        .page(params[:page])
+        .per(ITEMS_PER_PAGE)
+        .order(:label)
+    end
+  end
+end
diff --git a/app/views/instructeurs/groupe_instructeurs/index.html.haml b/app/views/instructeurs/groupe_instructeurs/index.html.haml
new file mode 100644
index 000000000..66f3369b0
--- /dev/null
+++ b/app/views/instructeurs/groupe_instructeurs/index.html.haml
@@ -0,0 +1,19 @@
+- content_for(:title, "Notifications pour #{@procedure.libelle}")
+
+= render partial: 'new_administrateur/breadcrumbs',
+  locals: { steps: [link_to(@procedure.libelle, procedure_path(@procedure)),
+                    'Groupes d’instructeurs'] }
+
+.container.groupe-instructeur
+  .card
+    .card-title Gestion des Groupes
+    %table.table.mt-2
+      %thead
+        %tr
+          %th{ colspan: 2 } Liste des groupes
+      %tbody
+        - @groupes_instructeurs.each do |group|
+          %tr
+            %td= group.label
+
+    = paginate @groupes_instructeurs
diff --git a/config/routes.rb b/config/routes.rb
index 5730cf8ef..7730df462 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -288,6 +288,8 @@ Rails.application.routes.draw do
   scope module: 'instructeurs', as: 'instructeur' do
     resources :procedures, only: [:index, :show], param: :procedure_id do
       member do
+        resources :groupes, only: [:index], controller: 'groupe_instructeurs'
+
         patch 'update_displayed_fields'
         get 'update_sort/:table/:column' => 'procedures#update_sort', as: 'update_sort'
         post 'add_filter'
diff --git a/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb b/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb
new file mode 100644
index 000000000..97aa23cef
--- /dev/null
+++ b/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb
@@ -0,0 +1,31 @@
+describe Instructeurs::GroupeInstructeursController, type: :controller do
+  render_views
+
+  let(:instructeur) { create(:instructeur) }
+  let(:procedure) { create(:procedure, :published) }
+  let!(:gi_1_1) { procedure.defaut_groupe_instructeur }
+  let!(:gi_1_2) { procedure.groupe_instructeurs.create(label: 'groupe instructeur 2') }
+
+  let(:procedure2) { create(:procedure, :published) }
+  let!(:gi_2_2) { procedure2.groupe_instructeurs.create(label: 'groupe instructeur 2 2') }
+
+  before do
+    gi_1_2.instructeurs << instructeur
+    sign_in(instructeur.user)
+  end
+
+  describe '#index' do
+    context 'of a procedure I own' do
+      before do
+        get :index, params: { procedure_id: procedure.id }
+      end
+
+      context 'when a procedure has multiple groups' do
+        it { expect(response).to have_http_status(:ok) }
+        it { expect(response.body).to include(gi_1_2.label) }
+        it { expect(response.body).not_to include(gi_1_1.label) }
+        it { expect(response.body).not_to include(gi_2_2.label) }
+      end
+    end
+  end
+end

From 94081a39977eda1ca282f1161a38c0e1674335a8 Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Thu, 24 Oct 2019 22:17:55 +0200
Subject: [PATCH 07/11] Show Groupe Instructeur

---
 .../groupe_instructeurs_controller.rb         | 18 ++++++++++++++++
 .../groupe_instructeurs/index.html.haml       |  1 +
 .../groupe_instructeurs/show.html.haml        | 21 +++++++++++++++++++
 config/routes.rb                              |  2 +-
 .../groupe_instructeurs_controller_spec.rb    |  8 +++++++
 5 files changed, 49 insertions(+), 1 deletion(-)
 create mode 100644 app/views/instructeurs/groupe_instructeurs/show.html.haml

diff --git a/app/controllers/instructeurs/groupe_instructeurs_controller.rb b/app/controllers/instructeurs/groupe_instructeurs_controller.rb
index bae72aa36..77098ad65 100644
--- a/app/controllers/instructeurs/groupe_instructeurs_controller.rb
+++ b/app/controllers/instructeurs/groupe_instructeurs_controller.rb
@@ -7,6 +7,12 @@ module Instructeurs
       @groupes_instructeurs = paginated_groupe_instructeurs
     end
 
+    def show
+      @procedure = procedure
+      @groupe_instructeur = groupe_instructeur
+      @instructeurs = paginated_instructeurs
+    end
+
     private
 
     def procedure
@@ -16,6 +22,10 @@ module Instructeurs
         .find(params[:procedure_id])
     end
 
+    def groupe_instructeur
+      current_instructeur.groupe_instructeurs.find(params[:id])
+    end
+
     def paginated_groupe_instructeurs
       current_instructeur
         .groupe_instructeurs
@@ -24,5 +34,13 @@ module Instructeurs
         .per(ITEMS_PER_PAGE)
         .order(:label)
     end
+
+    def paginated_instructeurs
+      groupe_instructeur
+        .instructeurs
+        .page(params[:page])
+        .per(ITEMS_PER_PAGE)
+        .order(:email)
+    end
   end
 end
diff --git a/app/views/instructeurs/groupe_instructeurs/index.html.haml b/app/views/instructeurs/groupe_instructeurs/index.html.haml
index 66f3369b0..97e62c876 100644
--- a/app/views/instructeurs/groupe_instructeurs/index.html.haml
+++ b/app/views/instructeurs/groupe_instructeurs/index.html.haml
@@ -15,5 +15,6 @@
         - @groupes_instructeurs.each do |group|
           %tr
             %td= group.label
+            %td.actions= link_to "voir", instructeur_groupe_path(@procedure, group)
 
     = paginate @groupes_instructeurs
diff --git a/app/views/instructeurs/groupe_instructeurs/show.html.haml b/app/views/instructeurs/groupe_instructeurs/show.html.haml
new file mode 100644
index 000000000..6cef25cc7
--- /dev/null
+++ b/app/views/instructeurs/groupe_instructeurs/show.html.haml
@@ -0,0 +1,21 @@
+- content_for(:title, "Instructeurs du groupe #{@groupe_instructeur.label}")
+
+= render partial: 'new_administrateur/breadcrumbs',
+  locals: { steps: [link_to(@procedure.libelle, procedure_path(@procedure)),
+                    link_to('Groupes d’instructeurs', instructeur_groupes_path(@procedure)),
+                    @groupe_instructeur.label] }
+
+.container.groupe-instructeur
+  %h1 Groupe « #{@groupe_instructeur.label} »
+
+  .card.mt-2
+    .card-title Gestion des instructeurs
+
+    %table.table.mt-2
+      %thead
+        %tr
+          %th{ colspan: 2 } Instructeurs affectés
+      %tbody
+        - @instructeurs.each do |instructeur|
+          %tr
+            %td= instructeur.email
diff --git a/config/routes.rb b/config/routes.rb
index 7730df462..e6c624acf 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -288,7 +288,7 @@ Rails.application.routes.draw do
   scope module: 'instructeurs', as: 'instructeur' do
     resources :procedures, only: [:index, :show], param: :procedure_id do
       member do
-        resources :groupes, only: [:index], controller: 'groupe_instructeurs'
+        resources :groupes, only: [:index, :show], controller: 'groupe_instructeurs'
 
         patch 'update_displayed_fields'
         get 'update_sort/:table/:column' => 'procedures#update_sort', as: 'update_sort'
diff --git a/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb b/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb
index 97aa23cef..ecc1b5ab6 100644
--- a/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb
+++ b/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb
@@ -28,4 +28,12 @@ describe Instructeurs::GroupeInstructeursController, type: :controller do
       end
     end
   end
+
+  describe '#show' do
+    context 'of a group I belong to' do
+      before { get :show, params: { procedure_id: procedure.id, id: gi_1_2.id } }
+
+      it { expect(response).to have_http_status(:ok) }
+    end
+  end
 end

From 79b808470c284c55b6ad2783185b90025d9d9bdd Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Fri, 25 Oct 2019 09:26:31 +0200
Subject: [PATCH 08/11] Add instructeur

---
 .../groupe_instructeurs_controller.rb         | 32 +++++++++++++++++++
 .../groupe_instructeurs/show.html.haml        |  8 +++++
 config/routes.rb                              |  6 +++-
 .../groupe_instructeurs_controller_spec.rb    | 26 +++++++++++++++
 4 files changed, 71 insertions(+), 1 deletion(-)

diff --git a/app/controllers/instructeurs/groupe_instructeurs_controller.rb b/app/controllers/instructeurs/groupe_instructeurs_controller.rb
index 77098ad65..a2df503b3 100644
--- a/app/controllers/instructeurs/groupe_instructeurs_controller.rb
+++ b/app/controllers/instructeurs/groupe_instructeurs_controller.rb
@@ -13,8 +13,36 @@ module Instructeurs
       @instructeurs = paginated_instructeurs
     end
 
+    def add_instructeur
+      @instructeur = Instructeur.find_by(email: instructeur_email) ||
+        create_instructeur(instructeur_email)
+
+      if groupe_instructeur.instructeurs.include?(@instructeur)
+        flash[:alert] = "L’instructeur « #{instructeur_email} » est déjà dans le groupe."
+
+      else
+        groupe_instructeur.instructeurs << @instructeur
+        flash[:notice] = "L’instructeur « #{instructeur_email} » a été affecté au groupe."
+        GroupeInstructeurMailer
+          .add_instructeur(groupe_instructeur, @instructeur, current_user.email)
+          .deliver_later
+      end
+
+      redirect_to instructeur_groupe_path(procedure, groupe_instructeur)
+    end
+
     private
 
+    def create_instructeur(email)
+      user = User.create_or_promote_to_instructeur(
+        email,
+        SecureRandom.hex,
+        administrateurs: [procedure.administrateurs.first]
+      )
+      user.invite!
+      user.instructeur
+    end
+
     def procedure
       current_instructeur
         .procedures
@@ -42,5 +70,9 @@ module Instructeurs
         .per(ITEMS_PER_PAGE)
         .order(:email)
     end
+
+    def instructeur_email
+      params[:instructeur][:email].strip.downcase
+    end
   end
 end
diff --git a/app/views/instructeurs/groupe_instructeurs/show.html.haml b/app/views/instructeurs/groupe_instructeurs/show.html.haml
index 6cef25cc7..d2a363451 100644
--- a/app/views/instructeurs/groupe_instructeurs/show.html.haml
+++ b/app/views/instructeurs/groupe_instructeurs/show.html.haml
@@ -10,6 +10,14 @@
 
   .card.mt-2
     .card-title Gestion des instructeurs
+    = form_for :instructeur,
+      url: { action: :add_instructeur },
+      html: { class: 'form' } do |f|
+
+      = f.label :email do
+        Affecter un nouvel instructeur
+      = f.email_field :email, placeholder: 'marie.dupont@exemple.fr', required: true
+      = f.submit 'Affecter', class: 'button primary send'
 
     %table.table.mt-2
       %thead
diff --git a/config/routes.rb b/config/routes.rb
index e6c624acf..192b5937f 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -288,7 +288,11 @@ Rails.application.routes.draw do
   scope module: 'instructeurs', as: 'instructeur' do
     resources :procedures, only: [:index, :show], param: :procedure_id do
       member do
-        resources :groupes, only: [:index, :show], controller: 'groupe_instructeurs'
+        resources :groupes, only: [:index, :show], controller: 'groupe_instructeurs' do
+          member do
+            post 'add_instructeur'
+          end
+        end
 
         patch 'update_displayed_fields'
         get 'update_sort/:table/:column' => 'procedures#update_sort', as: 'update_sort'
diff --git a/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb b/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb
index ecc1b5ab6..1a05a55ac 100644
--- a/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb
+++ b/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb
@@ -36,4 +36,30 @@ describe Instructeurs::GroupeInstructeursController, type: :controller do
       it { expect(response).to have_http_status(:ok) }
     end
   end
+
+  describe '#add_instructeur' do
+    before do
+      post :add_instructeur,
+        params: {
+          procedure_id: procedure.id,
+          id: gi_1_2.id,
+          instructeur: { email: new_instructeur_email }
+        }
+    end
+
+    context 'of a new instructeur' do
+      let(:new_instructeur_email) { 'new_instructeur@mail.com' }
+
+      it { expect(gi_1_2.instructeurs.pluck(:email)).to include(new_instructeur_email) }
+      it { expect(flash.notice).to be_present }
+      it { expect(response).to redirect_to(instructeur_groupe_path(procedure, gi_1_2)) }
+    end
+
+    context 'of an instructeur already in the group' do
+      let(:new_instructeur_email) { instructeur.email }
+
+      it { expect(flash.alert).to be_present }
+      it { expect(response).to redirect_to(instructeur_groupe_path(procedure, gi_1_2)) }
+    end
+  end
 end

From 477f7c983724b3ad2192ea771a98d349dc7abac8 Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Fri, 25 Oct 2019 10:17:39 +0200
Subject: [PATCH 09/11] Remove instructeur

---
 .../groupe_instructeurs_controller.rb         | 20 +++++++++++
 .../groupe_instructeurs/show.html.haml        |  8 +++++
 config/routes.rb                              |  1 +
 .../groupe_instructeurs_controller_spec.rb    | 35 +++++++++++++++++++
 4 files changed, 64 insertions(+)

diff --git a/app/controllers/instructeurs/groupe_instructeurs_controller.rb b/app/controllers/instructeurs/groupe_instructeurs_controller.rb
index a2df503b3..0e5a3b7e0 100644
--- a/app/controllers/instructeurs/groupe_instructeurs_controller.rb
+++ b/app/controllers/instructeurs/groupe_instructeurs_controller.rb
@@ -31,6 +31,22 @@ module Instructeurs
       redirect_to instructeur_groupe_path(procedure, groupe_instructeur)
     end
 
+    def remove_instructeur
+      if groupe_instructeur.instructeurs.one?
+        flash[:alert] = "Suppression impossible : il doit y avoir au moins un instructeur dans le groupe"
+
+      else
+        @instructeur = Instructeur.find(instructeur_id)
+        groupe_instructeur.instructeurs.destroy(@instructeur)
+        flash[:notice] = "L’instructeur « #{@instructeur.email} » a été retiré du groupe."
+        GroupeInstructeurMailer
+          .remove_instructeur(groupe_instructeur, @instructeur, current_user.email)
+          .deliver_later
+      end
+
+      redirect_to instructeur_groupe_path(procedure, groupe_instructeur)
+    end
+
     private
 
     def create_instructeur(email)
@@ -74,5 +90,9 @@ module Instructeurs
     def instructeur_email
       params[:instructeur][:email].strip.downcase
     end
+
+    def instructeur_id
+      params[:instructeur][:id]
+    end
   end
 end
diff --git a/app/views/instructeurs/groupe_instructeurs/show.html.haml b/app/views/instructeurs/groupe_instructeurs/show.html.haml
index d2a363451..1fcc742ef 100644
--- a/app/views/instructeurs/groupe_instructeurs/show.html.haml
+++ b/app/views/instructeurs/groupe_instructeurs/show.html.haml
@@ -27,3 +27,11 @@
         - @instructeurs.each do |instructeur|
           %tr
             %td= instructeur.email
+            %td.actions= button_to 'retirer',
+              { action: :remove_instructeur },
+              { method: :delete,
+                data: { confirm: "Êtes-vous sûr de vouloir retirer l’instructeur « #{instructeur.email} » du groupe  « #{@groupe_instructeur.label} » ?" },
+                params: { instructeur: { id: instructeur.id }},
+                class: 'button' }
+
+    = paginate @instructeurs
diff --git a/config/routes.rb b/config/routes.rb
index 192b5937f..16b257416 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -291,6 +291,7 @@ Rails.application.routes.draw do
         resources :groupes, only: [:index, :show], controller: 'groupe_instructeurs' do
           member do
             post 'add_instructeur'
+            delete 'remove_instructeur'
           end
         end
 
diff --git a/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb b/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb
index 1a05a55ac..c10912227 100644
--- a/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb
+++ b/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb
@@ -62,4 +62,39 @@ describe Instructeurs::GroupeInstructeursController, type: :controller do
       it { expect(response).to redirect_to(instructeur_groupe_path(procedure, gi_1_2)) }
     end
   end
+
+  describe '#remove_instructeur' do
+    let!(:new_instructeur) { create(:instructeur) }
+
+    before { gi_1_1.instructeurs << instructeur << new_instructeur }
+
+    def remove_instructeur(instructeur)
+      delete :remove_instructeur,
+        params: {
+          procedure_id: procedure.id,
+          id: gi_1_1.id,
+          instructeur: { id: instructeur.id }
+        }
+    end
+
+    context 'when there are many instructeurs' do
+      before { remove_instructeur(new_instructeur) }
+
+      it { expect(gi_1_1.instructeurs).to include(instructeur) }
+      it { expect(gi_1_1.reload.instructeurs.count).to eq(1) }
+      it { expect(response).to redirect_to(instructeur_groupe_path(procedure, gi_1_1)) }
+    end
+
+    context 'when there is only one instructeur' do
+      before do
+        remove_instructeur(new_instructeur)
+        remove_instructeur(instructeur)
+      end
+
+      it { expect(gi_1_1.instructeurs).to include(instructeur) }
+      it { expect(gi_1_1.instructeurs.count).to eq(1) }
+      it { expect(flash.alert).to eq('Suppression impossible : il doit y avoir au moins un instructeur dans le groupe') }
+      it { expect(response).to redirect_to(instructeur_groupe_path(procedure, gi_1_1)) }
+    end
+  end
 end

From 652e95d0c5acb838e89fc1778f9e88d28a3eb5e4 Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Mon, 28 Oct 2019 15:28:14 +0100
Subject: [PATCH 10/11] Add link from instructeur procedure view

---
 app/views/instructeurs/procedures/show.html.haml | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/app/views/instructeurs/procedures/show.html.haml b/app/views/instructeurs/procedures/show.html.haml
index 302ce3a95..229336345 100644
--- a/app/views/instructeurs/procedures/show.html.haml
+++ b/app/views/instructeurs/procedures/show.html.haml
@@ -13,6 +13,12 @@
         |
         = link_to 'statistiques', stats_instructeur_procedure_path(@procedure), class: 'header-link', data: { turbolinks: false } # Turbolinks disabled for Chartkick. See Issue #350
 
+        - if @procedure.routee?
+          |
+          - if current_administrateur.present? && current_administrateur.owns?(@procedure)
+            = link_to 'instructeurs', procedure_groupe_instructeurs_path(@procedure), class: 'header-link'
+          - else
+            = link_to 'instructeurs', instructeur_groupes_path(@procedure), class: 'header-link'
 
         %ul.tabs
           = tab_item('à suivre',

From 73d4ecf35dbe9b906eaf3418a05f13120fd012ca Mon Sep 17 00:00:00 2001
From: Paul Chavard <git@paul.chavard.net>
Date: Wed, 30 Oct 2019 16:15:38 +0100
Subject: [PATCH 11/11] Add a DS_PROXY_URL env variable

---
 config/env.example                    | 1 +
 config/environments/development.rb    | 2 +-
 config/initializers/active_storage.rb | 2 +-
 3 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/config/env.example b/config/env.example
index 84fd7aeb7..4f704adab 100644
--- a/config/env.example
+++ b/config/env.example
@@ -24,6 +24,7 @@ FOG_OPENSTACK_REGION=""
 FOG_DIRECTORY=""
 FOG_ENABLED=""
 CARRIERWAVE_CACHE_DIR="/tmp/tps-local-cache"
+DS_PROXY_URL=""
 
 FC_PARTICULIER_ID=""
 FC_PARTICULIER_SECRET=""
diff --git a/config/environments/development.rb b/config/environments/development.rb
index 2a5367e65..385185b36 100644
--- a/config/environments/development.rb
+++ b/config/environments/development.rb
@@ -22,7 +22,7 @@ Rails.application.configure do
   # Don't care if the mailer can't send.
   config.action_mailer.raise_delivery_errors = false
 
-  config.active_storage.service = :local
+  config.active_storage.service = ENV['FOG_ENABLED'] == 'enabled' ? :openstack : :local
 
   # Print deprecation notices to the Rails logger.
   config.active_support.deprecation = :log
diff --git a/config/initializers/active_storage.rb b/config/initializers/active_storage.rb
index 1d600d89c..dd4d57d48 100644
--- a/config/initializers/active_storage.rb
+++ b/config/initializers/active_storage.rb
@@ -32,7 +32,7 @@ module Fog::OpenStack::Auth::Catalog
 
     def publicize(url)
       search = %r{^https://[^/]+/}
-      replace = 'https://static.demarches-simplifiees.fr/'
+      replace = "#{ENV['DS_PROXY_URL']}/"
       url.gsub(search, replace)
     end
   end