-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Modified the code in order to support with IS 5.3.0 #16
Conversation
@@ -39,7 +39,7 @@ | |||
</init-param> | |||
<init-param> | |||
<param-name>serverName</param-name> | |||
<param-value>https://localhost:8080</param-value> | |||
<param-value>https://localhost:8443</param-value> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please explain the reasons for changing this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we use 8080, https won't work on Mac.
import org.wso2.carbon.registry.core.utils.UUIDGenerator; | ||
|
||
import java.io.UnsupportedEncodingException; | ||
import java.io.*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shall we avoid the *
serviceUrl = decoder | ||
.decode(casMessageContext.getRequest().getQueryString().replace("service=", ""), "UTF-8"); | ||
} catch (UnsupportedEncodingException e) { | ||
throw new FrameworkException("Unable to decode the service URL", e); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we verify with IS team if this is the best way to do this?
import org.wso2.carbon.identity.sso.cas.exception.CAS2ClientException; | ||
import org.wso2.carbon.identity.sso.cas.util.CASResourceReader; | ||
import org.wso2.carbon.identity.sso.cas.util.CASSSOUtil; | ||
|
||
import javax.servlet.http.HttpServletRequest; | ||
import javax.servlet.http.HttpServletResponse; | ||
import java.io.UnsupportedEncodingException; | ||
import java.io.*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Try to avoid * in import
} else { | ||
builder = new CASServiceValidateRequest.CASServiceValidateRequestBuilder(request, response); | ||
((CASServiceValidateRequest.CASServiceValidateRequestBuilder) builder).setLocale(request); | ||
} | ||
} else { | ||
throw CAS2ClientException.error("Invalid request message or invalid service url"); | ||
} | ||
|
||
Enumeration attributeNames = request.getAttributeNames(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With this are we removing needed elements as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We remove non serialised attributes for now. Once they fix the issue in framework level, we need to remove this part from the authenticator.
No description provided.