Re: [Dev] [UUF] Pet store sample not working without last forwards slash

2016-05-26 Thread SajithAR Ariyarathna
>
> But cannot be accessed with "http://127.0.0.1:8080/pets-store; (with the
> missing forward slash at the end). I get a "Cannot find an app for
> context 'null'." message on the browser.
>
This is a bug. When URL is "http://127.0.0.1:8080/pets-store;, then URI
becomes "/pets-store". UUF assumes an URI has at least 02 slashes [1].
Will fix this.

[1]
https://github.com/wso2/carbon-uuf/blob/v1.0.0-m5/uuf-core/src/main/java/org/wso2/carbon/uuf/internal/util/RequestUtil.java#L51

On Thu, May 26, 2016 at 7:26 PM, Manuranga Perera  wrote:

> it was working on old release
> https://github.com/wso2/carbon-uuf/blob/v1.0.0-m2/uuf-core/src/main/java/org/wso2/carbon/uuf/UUFRegistry.java#L82
> but some how it has been removed in new, we should add it back
>
> On Thu, May 26, 2016 at 9:17 AM, Sudharma Subasinghe 
> wrote:
>
>> Hi Hemika,
>>
>> I faced the same problem. I think, it is because the implementation says
>> "An URI must end with a slash."
>>
>> [1]
>> https://github.com/wso2/carbon-uuf/blob/v1.0.0-m5/uuf-core/src/main/java/org/wso2/carbon/uuf/internal/util/RequestUtil.java#L29
>>
>> On Thu, May 26, 2016 at 6:39 PM, Manuranga Perera  wrote:
>>
>>> Actually without slash one should redirect to the other , that's what
>>> tomcat does
>>>
>>> ___
>>> Dev mailing list
>>> Dev@wso2.org
>>> http://wso2.org/cgi-bin/mailman/listinfo/dev
>>>
>>>
>>
>>
>> --
>> Sudharma Subasinghe,
>> Software Engineer,
>> WSO2 Inc.
>> Email: sudhar...@wso2.com 
>> Mobile : +94 710 565 157 <%2B94%20718%20210%20200>
>>
>> ___
>> Dev mailing list
>> Dev@wso2.org
>> http://wso2.org/cgi-bin/mailman/listinfo/dev
>>
>>
>
>
> --
> With regards,
> *Manu*ranga Perera.
>
> phone : 071 7 70 20 50
> mail : m...@wso2.com
>
> ___
> Dev mailing list
> Dev@wso2.org
> http://wso2.org/cgi-bin/mailman/listinfo/dev
>
>


-- 
Sajith Janaprasad Ariyarathna
Software Engineer; WSO2, Inc.;  http://wso2.com/
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev


Re: [Dev] [UUF] Pet store sample not working without last forwards slash

2016-05-26 Thread Manuranga Perera
it was working on old release
https://github.com/wso2/carbon-uuf/blob/v1.0.0-m2/uuf-core/src/main/java/org/wso2/carbon/uuf/UUFRegistry.java#L82
but some how it has been removed in new, we should add it back

On Thu, May 26, 2016 at 9:17 AM, Sudharma Subasinghe 
wrote:

> Hi Hemika,
>
> I faced the same problem. I think, it is because the implementation says
> "An URI must end with a slash."
>
> [1]
> https://github.com/wso2/carbon-uuf/blob/v1.0.0-m5/uuf-core/src/main/java/org/wso2/carbon/uuf/internal/util/RequestUtil.java#L29
>
> On Thu, May 26, 2016 at 6:39 PM, Manuranga Perera  wrote:
>
>> Actually without slash one should redirect to the other , that's what
>> tomcat does
>>
>> ___
>> Dev mailing list
>> Dev@wso2.org
>> http://wso2.org/cgi-bin/mailman/listinfo/dev
>>
>>
>
>
> --
> Sudharma Subasinghe,
> Software Engineer,
> WSO2 Inc.
> Email: sudhar...@wso2.com 
> Mobile : +94 710 565 157 <%2B94%20718%20210%20200>
>
> ___
> Dev mailing list
> Dev@wso2.org
> http://wso2.org/cgi-bin/mailman/listinfo/dev
>
>


-- 
With regards,
*Manu*ranga Perera.

phone : 071 7 70 20 50
mail : m...@wso2.com
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev


Re: [Dev] [UUF] Pet store sample not working without last forwards slash

2016-05-26 Thread Sudharma Subasinghe
Hi Hemika,

I faced the same problem. I think, it is because the implementation says
"An URI must end with a slash."

[1]
https://github.com/wso2/carbon-uuf/blob/v1.0.0-m5/uuf-core/src/main/java/org/wso2/carbon/uuf/internal/util/RequestUtil.java#L29

On Thu, May 26, 2016 at 6:39 PM, Manuranga Perera  wrote:

> Actually without slash one should redirect to the other , that's what
> tomcat does
>
> ___
> Dev mailing list
> Dev@wso2.org
> http://wso2.org/cgi-bin/mailman/listinfo/dev
>
>


-- 
Sudharma Subasinghe,
Software Engineer,
WSO2 Inc.
Email: sudhar...@wso2.com 
Mobile : +94 710 565 157 <%2B94%20718%20210%20200>
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev


Re: [Dev] [UUF] Pet store sample not working without last forwards slash

2016-05-26 Thread Manuranga Perera
Actually without slash one should redirect to the other , that's what
tomcat does
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev


Re: [Dev] [UUF] Pet store sample not working without last forwards slash

2016-05-26 Thread Hemika Kodikara
IMO, ideally yes.

Hemika Kodikara
Software Engineer
WSO2 Inc.
lean . enterprise . middleware
http://wso2.com

Mobile : +9477762

On Thu, May 26, 2016 at 5:40 PM, Ramith Jayasinghe  wrote:

> I think in both cases the app should load in the browser?
>
> On Thu, May 26, 2016 at 5:31 PM, Hemika Kodikara  wrote:
> > Hi All,
> >
> > The pets-store can be accessed with "http://127.0.0.1:8080/pets-store/;
> >
> > But cannot be accessed with "http://127.0.0.1:8080/pets-store; (with the
> > missing forward slash at the end). I get a "Cannot find an app for
> context
> > 'null'." message on the browser.
> >
> > Also a small suggestion - Can we show the URLs of the apps deployed on
> the
> > console when the server startsup. IMO, it will help out the users in user
> > experience wise.
> >
> > Regards,
> > Hemika
> >
> > Hemika Kodikara
> > Software Engineer
> > WSO2 Inc.
> > lean . enterprise . middleware
> > http://wso2.com
> >
> > Mobile : +9477762
> >
> > ___
> > Dev mailing list
> > Dev@wso2.org
> > http://wso2.org/cgi-bin/mailman/listinfo/dev
> >
>
>
>
> --
> Ramith Jayasinghe
> Technical Lead
> WSO2 Inc., http://wso2.com
> lean.enterprise.middleware
>
> E: ram...@wso2.com
> P: +94 772534930
>
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev


Re: [Dev] [UUF] Pet store sample not working without last forwards slash

2016-05-26 Thread Ramith Jayasinghe
I think in both cases the app should load in the browser?

On Thu, May 26, 2016 at 5:31 PM, Hemika Kodikara  wrote:
> Hi All,
>
> The pets-store can be accessed with "http://127.0.0.1:8080/pets-store/;
>
> But cannot be accessed with "http://127.0.0.1:8080/pets-store; (with the
> missing forward slash at the end). I get a "Cannot find an app for context
> 'null'." message on the browser.
>
> Also a small suggestion - Can we show the URLs of the apps deployed on the
> console when the server startsup. IMO, it will help out the users in user
> experience wise.
>
> Regards,
> Hemika
>
> Hemika Kodikara
> Software Engineer
> WSO2 Inc.
> lean . enterprise . middleware
> http://wso2.com
>
> Mobile : +9477762
>
> ___
> Dev mailing list
> Dev@wso2.org
> http://wso2.org/cgi-bin/mailman/listinfo/dev
>



-- 
Ramith Jayasinghe
Technical Lead
WSO2 Inc., http://wso2.com
lean.enterprise.middleware

E: ram...@wso2.com
P: +94 772534930
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev


[Dev] [UUF] Pet store sample not working without last forwards slash

2016-05-26 Thread Hemika Kodikara
Hi All,

The pets-store can be accessed with "http://127.0.0.1:8080/pets-store/;

But cannot be accessed with "http://127.0.0.1:8080/pets-store; (with the
missing forward slash at the end). I get a "Cannot find an app for context
'null'." message on the browser.

Also a small suggestion - Can we show the URLs of the apps deployed on the
console when the server startsup. IMO, it will help out the users in user
experience wise.

Regards,
Hemika

Hemika Kodikara
Software Engineer
WSO2 Inc.
lean . enterprise . middleware
http://wso2.com

Mobile : +9477762
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev