Merge pull request #1326 from ShanChathusanda93/consent-denied-url-fi…

…x-hybrid-branch Fix error response is not according to the spec in user denied consent for hybrid flow
wso2-extensions · Feb 20, 2020 · bf61808 · bf61808
2 parents f89935b + 8a34d6f
commit bf61808
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/...uth.endpoint/src/main/java/org/wso2/carbon/identity/oauth/endpoint/util/EndpointUtil.java b/...uth.endpoint/src/main/java/org/wso2/carbon/identity/oauth/endpoint/util/EndpointUtil.java
@@ -447,7 +447,8 @@ public static String getErrorRedirectURL(OAuthProblemException ex, OAuth2Paramet
         try {
             if (params != null) {
                 if (isNotBlank(params.getRedirectURI())) {
-                    if (OAuth2Util.isImplicitResponseType(params.getResponseType())) {
+                    if (OAuth2Util.isImplicitResponseType(params.getResponseType()) ||
+                            OAuth2Util.isHybridResponseType(params.getResponseType())) {
                         if (OAuthServerConfiguration.getInstance().isImplicitErrorFragment()) {
                             redirectURL = OAuthASResponse.errorResponse(HttpServletResponse.SC_FOUND)
                                     .error(ex).location(params.getRedirectURI())