springdoc.swagger-ui.oauth.additionalQueryStringParams should be map instead of list. fixes #567

Author: bnasslahsen

springdoc-openapi-common/src/main/java/org/springdoc/core/SpringDocPropertiesUtils.java

@@ -51,4 +51,10 @@ static void put(final String name, final String value, final Map<String, Object>
 		}
 	}
 
+	static void put(String name, Map<String,String> value, Map<String, Object> params) {
+		if (!CollectionUtils.isEmpty(value)) {
+			params.put(name, value);
+		}
+	}
+
 }

springdoc-openapi-common/src/main/java/org/springdoc/core/SwaggerUiOAuthProperties.java

@@ -19,7 +19,6 @@
 package org.springdoc.core;
 
 
-import java.util.List;
 import java.util.Map;
 import java.util.TreeMap;
 
@@ -51,7 +50,7 @@ public class SwaggerUiOAuthProperties {
 
 	private String scopeSeparator;
 
-	private List<String> additionalQueryStringParams;
+	private Map<String,String> additionalQueryStringParams;
 
 	private String useBasicAuthenticationWithAccessCodeGrant;
 
@@ -110,11 +109,11 @@ public void setScopeSeparator(String scopeSeparator) {
 		this.scopeSeparator = scopeSeparator;
 	}
 
-	public List<String> getAdditionalQueryStringParams() {
+	public Map<String, String> getAdditionalQueryStringParams() {
 		return additionalQueryStringParams;
 	}
 
-	public void setAdditionalQueryStringParams(List<String> additionalQueryStringParams) {
+	public void setAdditionalQueryStringParams(Map<String, String> additionalQueryStringParams) {
 		this.additionalQueryStringParams = additionalQueryStringParams;
 	}
 

springdoc-openapi-ui/src/test/java/test/org/springdoc/ui/app7/SpringDocApp7Test.java

@@ -30,7 +30,9 @@
 import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get;
 import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;
 
-@TestPropertySource(properties = "springdoc.swagger-ui.oauth.clientId=myClientId")
+@TestPropertySource(properties = { "springdoc.swagger-ui.oauth.clientId=myClientId",
+		"springdoc.swagger-ui.oauth.additionalQueryStringParams.test1=test1",
+		"springdoc.swagger-ui.oauth.additionalQueryStringParams.test2=test2" })
 public class SpringDocApp7Test extends AbstractSpringDocTest {
 
 	@Test

springdoc-openapi-ui/src/test/resources/results/app7

@@ -55,7 +55,7 @@
 
       window.ui = ui
 ui.initOAuth(
-{"clientId":"myClientId"})
+{"additionalQueryStringParams":{"test1":"test1","test2":"test2"},"clientId":"myClientId"})
     }
   </script>
   </body>