Merge pull request #146 from timlegge/testapp-changes

testapp: signed metadata messes with testing - make it an option

Author: timlegge

xt/testapp/lib/Saml2Test.pm

@@ -44,7 +44,7 @@ get '/' => sub {
         push @idps, \%tempidp;
     }
 
-    template 'index', { 'idps' => \@idps };
+    template 'index', { 'idps' => \@idps, 'sign_metadata' => config->{sign_metadata} };
 };
 
 get '/login' => sub {
@@ -253,9 +253,15 @@ post '/sls-post-response' => sub {
 };
 
 get '/metadata.xml' => sub {
+
     content_type 'application/octet-stream';
 
     my $sp = _sp();
+    if (defined params->{signmetadata} and params->{signmetadata} = 'on') {
+        $sp->{sign_metadata} = 1;
+    } else {
+        $sp->{sign_metadata} = 0;
+    }
     return $sp->metadata;
 };
 
@@ -290,7 +296,6 @@ sub _sp {
         org_display_name => config->{org_display_name},
         org_contact	 => config->{org_contact},
         authnreq_signed => config->{authnreq_signed},
-        sign_metadata => config->{sign_metadata},
     );
     return $sp;
 }

xt/testapp/views/index.tt

@@ -18,5 +18,22 @@
         </ol>
     <% end %>
 
-    <h2>Download SP <a href="/metadata.xml">Metadata</a></h2>
+    <h2>Download SP Metadata</h2>
+
+    <form action="/metadata.xml">
+        <div>
+            <button type="submit">Get Metadata</button>
+            <input
+              type="checkbox"
+              id="signmetadata"
+              name="signmetadata"
+              <% if sign_metadata == 1 %>
+                checked
+              <% end %>
+            />
+            <label for="signmetadata">Sign Metadata?</label>
+        </div>
+        <div>
+        </div>
+    </form>