UY-1453 revert state fixes

ppiernik · ppiernik · commit 673cd57dc75e · 2024-06-17T18:43:53.000+02:00
diff --git a/oauth/src/main/java/pl/edu/icm/unity/oauth/as/webauthz/OAuthAuthzWebEndpoint.java b/oauth/src/main/java/pl/edu/icm/unity/oauth/as/webauthz/OAuthAuthzWebEndpoint.java
@@ -233,9 +233,12 @@ public void doFilter(HttpServletRequest request, HttpServletResponse response) t
 		
 		private void returnOAuthError(HttpServletRequest request, HttpServletResponse response) throws IOException
 		{
+			String sessionId = request.getSession().getId();
+			log.debug(sessionId);
+			
 			OAuthAuthzContext ctx = OAuthSessionService.getContext(request).get();
 			AuthorizationErrorResponse oauthResponse = new AuthorizationErrorResponse(ctx.getReturnURI(),
-					OIDCError.LOGIN_REQUIRED,  State.parse(request.getParameter("state")),
+					OIDCError.LOGIN_REQUIRED,  ctx.getRequest().getState(),
 					ctx.getRequest().impliedResponseMode());
 			try
 			{

Original file line number	Diff line number	Diff line change
`@@ -233,9 +233,12 @@ public void doFilter(HttpServletRequest request, HttpServletResponse response) t`
`233`	`233`
`234`	`234`	`private void returnOAuthError(HttpServletRequest request, HttpServletResponse response) throws IOException`
`235`	`235`	`{`
	`236`	`+ String sessionId = request.getSession().getId();`
	`237`	`+ log.debug(sessionId);`
	`238`	`+`
`236`	`239`	`OAuthAuthzContext ctx = OAuthSessionService.getContext(request).get();`
`237`	`240`	`AuthorizationErrorResponse oauthResponse = new AuthorizationErrorResponse(ctx.getReturnURI(),`
`238`		`- OIDCError.LOGIN_REQUIRED, State.parse(request.getParameter("state")),`
	`241`	`+ OIDCError.LOGIN_REQUIRED, ctx.getRequest().getState(),`
`239`	`242`	`ctx.getRequest().impliedResponseMode());`
`240`	`243`	`try`
`241`	`244`	`{`