Hi Kasun,

I test my use-case with [1]. It works fine.

[1] - http://maven.wso2.org/nexus/content/repositories/
snapshots/org/wso2/is/wso2is/5.2.0-SNAPSHOT/

Thanks,

Biruntha

Associate Software Engineer
WSO2
Email : birun...@wso2.com
Linkedin : https://lk.linkedin.com/in/biruntha
Mobile : +94773718986

On Tue, Aug 23, 2016 at 6:27 PM, Kasun Bandara <kas...@wso2.com> wrote:

> Hi Biruntha,
>
> This issue has been already fixed in the public branch [1]. You can verify
> your use-case with the latest pack available in [2]
>
> Thanks,
> Kasun.
>
> [1] https://github.com/wso2-extensions/identity-inbound-
> auth-saml/blob/master/components/org.wso2.carbon.
> identity.sso.saml/src/main/java/org/wso2/carbon/identity/sso/saml/servlet/
> SAMLSSOProviderServlet.java
>
> [2] http://maven.wso2.org/nexus/content/repositories/
> snapshots/org/wso2/is/wso2is/5.2.0-SNAPSHOT/
>
> Thanks,
> Kasun
>
> On Tue, Aug 23, 2016 at 5:43 PM, Kasun Bandara <kas...@wso2.com> wrote:
>
>> Hi Biruntha,
>>
>> Thanks for fixing the issue. Can you please send the relevant PR to [1]
>> extension. We don't use the carbon-identity repository now.
>>
>> Thanks,
>> Kasun.
>>
>> [1] https://github.com/wso2-extensions/identity-inbound-auth-
>> saml/blob/master/components/org.wso2.carbon.identity.sso.
>> saml/src/main/java/org/wso2/carbon/identity/sso/saml/
>> servlet/SAMLSSOProviderServlet.java
>>
>> On Tue, Aug 23, 2016 at 4:48 PM, Johann Nallathamby <joh...@wso2.com>
>> wrote:
>>
>>>
>>>
>>> On Tue, Aug 23, 2016 at 4:14 PM, Biruntha Gnaneswaran <birun...@wso2.com
>>> > wrote:
>>>
>>>> Hi,
>>>>
>>>> I created jira [1] and fixed in [2].
>>>>
>>>> @ IS team please verify and merge.
>>>>
>>>> [1] - https://wso2.org/jira/browse/IDENTITY-5027
>>>>
>>>> [2] - https://github.com/wso2/carbon-identity/pull/1748
>>>>
>>>> Thanks,
>>>>
>>>> Biruntha
>>>>
>>>> Associate Software Engineer
>>>> WSO2
>>>> Email : birun...@wso2.com
>>>> Linkedin : https://lk.linkedin.com/in/biruntha
>>>> Mobile : +94773718986
>>>>
>>>> On Tue, Aug 23, 2016 at 6:05 AM, Malaka Silva <mal...@wso2.com> wrote:
>>>>
>>>>> Hi Biruntha,
>>>>>
>>>>> Good investigation on this matter. However we are not sure the impact
>>>>> of this change. Can you create a jira [1] and add the PR so that IS team
>>>>> can verify this in coming releases?
>>>>>
>>>>> [1] https://wso2.org/jira/browse/IDENTITY
>>>>>
>>>>> On Mon, Aug 22, 2016 at 6:04 PM, Biruntha Gnaneswaran <
>>>>> birun...@wso2.com> wrote:
>>>>>
>>>>>> Hi All,
>>>>>>
>>>>>> While posting a form from Mepin Authenticator to Authentication
>>>>>> endpoint, [1] will be executed. So it goes to [2]. From that method, it
>>>>>> only support for url redirecting not support for form posting. So, I
>>>>>> replace the logic [3] by [4]. Now it is working fine .So, To support post
>>>>>> request in authentication endpoint, we need to modify sso-saml module.
>>>>>>
>>>>>> [1] - https://github.com/wso2/carbon-identity/blob/master/componen
>>>>>> ts/sso-saml/org.wso2.carbon.identity.sso.saml/src/main/java/
>>>>>> org/wso2/carbon/identity/sso/saml/servlet/SAMLSSOProviderSer
>>>>>> vlet.java#L157-#L161
>>>>>>
>>>>>> [2] - https://github.com/wso2/carbon-identity/blob/master/componen
>>>>>> ts/sso-saml/org.wso2.carbon.identity.sso.saml/src/main/java/
>>>>>> org/wso2/carbon/identity/sso/saml/servlet/SAMLSSOProviderSer
>>>>>> vlet.java#L980
>>>>>>
>>>>>> [3] - https://github.com/wso2/carbon-identity/blob/master/componen
>>>>>> ts/sso-saml/org.wso2.carbon.identity.sso.saml/src/main/java/
>>>>>> org/wso2/carbon/identity/sso/saml/servlet/SAMLSSOProviderSer
>>>>>> vlet.java#L992
>>>>>>
>>>>>> [4] - https://github.com/wso2/carbon-identity/blob/master/componen
>>>>>> ts/sso-saml/org.wso2.carbon.identity.sso.saml/src/main/java/
>>>>>> org/wso2/carbon/identity/sso/saml/servlet/SAMLSSOProviderSer
>>>>>> vlet.java#L1031-#L1035
>>>>>>
>>>>>>
>>>>>> Thanks,
>>>>>>
>>>>>> Biruntha
>>>>>>
>>>>>> Associate Software Engineer
>>>>>> WSO2
>>>>>> Email : birun...@wso2.com
>>>>>> Linkedin : https://lk.linkedin.com/in/biruntha
>>>>>> Mobile : +94773718986
>>>>>>
>>>>>> On Fri, Aug 12, 2016 at 10:18 PM, Biruntha Gnaneswaran <
>>>>>> birun...@wso2.com> wrote:
>>>>>>
>>>>>>> Hi All,
>>>>>>>
>>>>>>>
>>>>>>> To solve the issue in $subject, i tried to post a html form from
>>>>>>> Authenticator to mepin.jsp using [1]. But still i couldn't solve that
>>>>>>> issue. It's again redirected to [2].
>>>>>>> Can anyone help me to solve this issue?
>>>>>>>
>>>>>>> [1]
>>>>>>>
>>>>>>> response.setContentType(*"text/html"*);java.io.PrintWriter out = 
>>>>>>> response.getWriter();
>>>>>>> out.println(*"<html><head>"*);out.println(*"</head><body>"*);out.println(*"<form
>>>>>>>  name=**\"**loginform**\"* *method=**\"**post**\"* 
>>>>>>> *action=**\"**/mepinauthenticationendpoint/mepin.jsp**\"**>"*);out.println(*"<input
>>>>>>>  type=**\"**hidden**\"* *id=**\"**param1**\"* *name=**\"**param1**\"* 
>>>>>>> *value=**\"**"*+param1+*"**\"**/>"*);out.println(*"<input 
>>>>>>> type=**\"**hidden**\"* *id=**\"**param2**\"* *name=**\"**param2**\"* 
>>>>>>> *value=**\"**"*+param2+*"**\"**/>"*);out.println(*"</form></body></html>"*);
>>>>>>>
>>>>>>> [2] https://localhost:9443/samlsso
>>>>>>>
>>>>>>> Thanks,
>>>>>>>
>>>>>>> Biruntha
>>>>>>>
>>>>>>> Associate Software Engineer
>>>>>>> WSO2
>>>>>>> Email : birun...@wso2.com
>>>>>>> Linkedin : https://lk.linkedin.com/in/biruntha
>>>>>>> Mobile : +94773718986
>>>>>>>
>>>>>>> On Fri, Jul 29, 2016 at 11:46 PM, Biruntha Gnaneswaran <
>>>>>>> birun...@wso2.com> wrote:
>>>>>>>
>>>>>>>> Hi Devs,
>>>>>>>>
>>>>>>>> In Mepin authenticator, some URL parameters are send to Mepin UI
>>>>>>>> page using GET method [1]. When I try to send these parameters via POST
>>>>>>>> method using HttpURLConnection class [2], it redirects to [3] not
>>>>>>>> to the actual Mepin UI page. While I try, I got the following Log .
>>>>>>>>
>>>>>>>> [2016-07-29 23:30:00,903] DEBUG {org.wso2.carbon.identity.appl
>>>>>>>> ication.authentication.framework.handler.request.impl.DefaultAuthenticationRequestHandler}
>>>>>>>> - In authentication flow
>>>>>>>>
>>>>>>>> [2016-07-29 23:30:00,903] DEBUG {org.wso2.carbon.identity.appl
>>>>>>>> ication.authentication.framework.handler.sequence.impl.DefaultStepBasedSequenceHandler}
>>>>>>>> - Executing the Step Based Authentication...
>>>>>>>>
>>>>>>>> [2016-07-29 23:30:00,903] DEBUG {org.wso2.carbon.identity.appl
>>>>>>>> ication.authentication.framework.handler.sequence.impl.DefaultStepBasedSequenceHandler}
>>>>>>>> - Starting Step: 1
>>>>>>>>
>>>>>>>> [2016-07-29 23:30:00,903] DEBUG {org.wso2.carbon.identity.appl
>>>>>>>> ication.authentication.framework.util.FrameworkUtils} - Finding
>>>>>>>> already authenticated IdPs of the Step
>>>>>>>>
>>>>>>>> [2016-07-29 23:30:00,903] DEBUG {org.wso2.carbon.identity.appl
>>>>>>>> ication.authentication.framework.handler.step.impl.DefaultStepHandler}
>>>>>>>> - Receive a response from the external party
>>>>>>>>
>>>>>>>> [2016-07-29 23:30:00,903] DEBUG {org.wso2.carbon.identity.appl
>>>>>>>> ication.authentication.framework.handler.step.impl.DefaultStepHandler}
>>>>>>>> - BasicAuthenticator can handle the request.
>>>>>>>>
>>>>>>>> [2016-07-29 23:30:00,929] DEBUG {org.wso2.carbon.identity.appl
>>>>>>>> ication.authentication.framework.handler.step.impl.DefaultStepHandler}
>>>>>>>> - BasicAuthenticator returned: SUCCESS_COMPLETED
>>>>>>>>
>>>>>>>> [2016-07-29 23:30:00,930] DEBUG {org.wso2.carbon.identity.appl
>>>>>>>> ication.authentication.framework.handler.sequence.impl.DefaultStepBasedSequenceHandler}
>>>>>>>> - Step 1 is completed. Going to get the next one.
>>>>>>>>
>>>>>>>> [2016-07-29 23:30:00,930] DEBUG {org.wso2.carbon.identity.appl
>>>>>>>> ication.authentication.framework.handler.sequence.impl.DefaultStepBasedSequenceHandler}
>>>>>>>> - Starting Step: 2
>>>>>>>>
>>>>>>>> [2016-07-29 23:30:00,930] DEBUG {org.wso2.carbon.identity.appl
>>>>>>>> ication.authentication.framework.util.FrameworkUtils} - Finding
>>>>>>>> already authenticated IdPs of the Step
>>>>>>>>
>>>>>>>> [2016-07-29 23:30:00,930] DEBUG {org.wso2.carbon.identity.appl
>>>>>>>> ication.authentication.framework.handler.step.impl.DefaultStepHandler}
>>>>>>>> - Step contains only a single IdP. Going to call it directly
>>>>>>>>
>>>>>>>> [2016-07-29 23:30:00,930] DEBUG {org.wso2.carbon.identity.appl
>>>>>>>> ication.authentication.framework.config.ConfigurationFacade} -
>>>>>>>> Trying to find the IdP for name: mepin
>>>>>>>>
>>>>>>>> [2016-07-29 23:30:00,934] DEBUG {org.wso2.carbon.identity.appl
>>>>>>>> ication.authentication.framework.config.ConfigurationFacade} - A
>>>>>>>> registered IdP was found
>>>>>>>>
>>>>>>>> [2016-07-29 23:30:03,520] DEBUG {org.wso2.carbon.identity.appl
>>>>>>>> ication.authentication.framework.handler.step.impl.DefaultStepHandler}
>>>>>>>> - MePINAuthenticator returned: INCOMPLETE
>>>>>>>>
>>>>>>>> [2016-07-29 23:30:03,521] DEBUG {org.wso2.carbon.identity.appl
>>>>>>>> ication.authentication.framework.handler.step.impl.DefaultStepHandler}
>>>>>>>> - MePINAuthenticator is redirecting
>>>>>>>>
>>>>>>>> [2016-07-29 23:30:03,521] DEBUG {org.wso2.carbon.identity.appl
>>>>>>>> ication.authentication.framework.handler.sequence.impl.DefaultStepBasedSequenceHandler}
>>>>>>>> - Step is not complete yet. Redirecting to outside.
>>>>>>>>
>>>>>>>>
>>>>>>>> So, I had offline chat with Dulanja. He said that it is not a
>>>>>>>> direct method when using POST method. POST method is not really
>>>>>>>> needed. Because, even facebook authenticator uses GET method to send 
>>>>>>>> URL
>>>>>>>> parameters. And also he give some suggestion,
>>>>>>>>
>>>>>>>>    -
>>>>>>>>
>>>>>>>>    For example, To send parameter called username, instead of
>>>>>>>>    using username as a name we can use “a” or “A” so from outside
>>>>>>>>    it can’t be guess.
>>>>>>>>    -
>>>>>>>>
>>>>>>>>    From java class , send as HTML body. While on submitting, need
>>>>>>>>    to get url parameters.
>>>>>>>>
>>>>>>>> @ Dulanja, Please add If I miss anything.
>>>>>>>>
>>>>>>>>
>>>>>>>> Your comments and suggestions are highly appreciated.
>>>>>>>>
>>>>>>>> [1]
>>>>>>>> *https://github.com/wso2-extensions/identity-outbound-auth-mepin/blob/master/component/authenticator/src/main/java/org/wso2/carbon/identity/authenticator/mepin/MepinAuthenticator.java#L112-#L116*
>>>>>>>> <https://github.com/wso2-extensions/identity-outbound-auth-mepin/blob/master/component/authenticator/src/main/java/org/wso2/carbon/identity/authenticator/mepin/MepinAuthenticator.java#L112-%23L116>
>>>>>>>>
>>>>>>>> [2] http://stackoverflow.com/questions/4205980/java-sending-http
>>>>>>>> -parameters-via-post-method-easily
>>>>>>>>
>>>>>>>> [3] https://localhost:9443/samlsso
>>>>>>>>
>>>>>>>>
>>>>>>>> Thanks,
>>>>>>>>
>>>>>>>>
>>>>>>>> Biruntha
>>>>>>>>
>>>>>>>> Associate Software Engineer
>>>>>>>> WSO2
>>>>>>>> Email : birun...@wso2.com
>>>>>>>> Linkedin : https://lk.linkedin.com/in/biruntha
>>>>>>>> Mobile : +94773718986
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>>
>>>>> Best Regards,
>>>>>
>>>>> Malaka Silva
>>>>> Senior Technical Lead
>>>>> M: +94 777 219 791
>>>>> Tel : 94 11 214 5345
>>>>> Fax :94 11 2145300
>>>>> Skype : malaka.sampath.silva
>>>>> LinkedIn : http://www.linkedin.com/pub/malaka-silva/6/33/77
>>>>> Blog : http://mrmalakasilva.blogspot.com/
>>>>>
>>>>> WSO2, Inc.
>>>>> lean . enterprise . middleware
>>>>> https://wso2.com/signature
>>>>> http://www.wso2.com/about/team/malaka-silva/
>>>>> <http://wso2.com/about/team/malaka-silva/>
>>>>> https://store.wso2.com/store/
>>>>>
>>>>> Don't make Trees rare, we should keep them with care
>>>>>
>>>>
>>>>
>>>
>>>
>>> --
>>> Thanks & Regards,
>>>
>>> *Johann Dilantha Nallathamby*
>>> Technical Lead & Product Lead of WSO2 Identity Server
>>> Governance Technologies Team
>>> WSO2, Inc.
>>> lean.enterprise.middleware
>>>
>>> Mobile - *+94777776950*
>>> Blog - *http://nallaa.wordpress.com <http://nallaa.wordpress.com>*
>>>
>>
>>
>>
>> --
>> Kasun Bandara
>> *Software Engineer*
>> Mobile : +94 (0) 718 338 360
>> <%2B94%20%280%29%20773%20451194>
>> kas...@wso2.com <thili...@wso2.com>
>>
>
>
>
> --
> Kasun Bandara
> *Software Engineer*
> Mobile : +94 (0) 718 338 360
> <%2B94%20%280%29%20773%20451194>
> kas...@wso2.com <thili...@wso2.com>
>
_______________________________________________
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev

Reply via email to