[java-identity-provider] branch main updated: IDP-2137 - dumpconfig CLI not functional

[Scott Cantor]

This is an automated email from the git hooks/post-receive script.

scantor pushed a commit to branch main
in repository java-identity-provider.

View the commit online:
http://git.shibboleth.net/view/?p=java-identity-provider.git;a=commit;h=11f561d341fdcbef97403ca4a9c99ad4b39b4c58

The following commit(s) were added to refs/heads/main by this push:
     new 11f561d34 IDP-2137 - dumpconfig CLI not functional
11f561d34 is described below

commit 11f561d341fdcbef97403ca4a9c99ad4b39b4c58
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Wed Jul 5 09:30:44 2023 -0400

    IDP-2137 - dumpconfig CLI not functional
    
    https://shibboleth.atlassian.net/browse/IDP-2137
---
 .../src/main/java/net/shibboleth/idp/cli/DumpConfigArguments.java  | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/idp-cli/src/main/java/net/shibboleth/idp/cli/DumpConfigArguments.java b/idp-cli/src/main/java/net/shibboleth/idp/cli/DumpConfigArguments.java
index 512a4b569..6e45fb891 100644
--- a/idp-cli/src/main/java/net/shibboleth/idp/cli/DumpConfigArguments.java
+++ b/idp-cli/src/main/java/net/shibboleth/idp/cli/DumpConfigArguments.java
@@ -33,7 +33,7 @@ public class DumpConfigArguments extends AbstractCommandLineArguments {
     @Nullable private String requester;
 
     /** Index into metadata.  */
-    @Parameter(names = {"-P", "--profile"}, description = "Profile identifier")
+    @Parameter(names = {"-P", "--profile"}, required = true, description = "Profile identifier")
     @Nullable private String profile;
 
     /** Exact protocol for metadata lookup.  */
@@ -100,7 +100,9 @@ public class DumpConfigArguments extends AbstractCommandLineArguments {
         }
         
         try {
-            builder.append("requester=").append(URLEncoder.encode(requester, "UTF-8"));
+            builder.append("requester=").append(URLEncoder.encode(requester, "UTF-8"))
+                .append("&profile=").append(URLEncoder.encode(profile, "UTF-8"));
+            
             if (saml1) {
                 builder.append("&saml1");
             } else if (saml2) {
@@ -112,6 +114,7 @@ public class DumpConfigArguments extends AbstractCommandLineArguments {
             } else if (protocol != null) {
                 builder.append("&protocol=").append(URLEncoder.encode(protocol, "UTF-8"));
             }
+            
         } catch (final UnsupportedEncodingException e) {
             // UTF-8 is a required encoding. 
         }

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.