You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by bu...@apache.org on 2013/09/13 07:27:19 UTC

[Bug 55508] TldLocationsCache not recognising custom TLDs in WEB-INF/lib

https://issues.apache.org/bugzilla/show_bug.cgi?id=55508

Jeremy Boynes <jb...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|INVALID                     |---

--- Comment #4 from Jeremy Boynes <jb...@apache.org> ---
Reopening this for further discussion, and to see whether I need to change
TldScanner in trunk as I think I introduced a regression here.

I interpreted the spec as saying that all TLDs under /WEB-INF *must* be
included except for any under /WEB-INF/tags (except for any implicit.tld found
there). As a result, although applications *should* not place them in
/WEB-INF/lib or /WEB-INF/classes, that was a recommendation and if they were
there the container would need to include them.

The change to TldScanner in TC8 unintentionally resurrected the behaviour from
TC6 that was removed in TC7.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Re: [Bug 55508] TldLocationsCache not recognising custom TLDs in WEB-INF/lib

Posted by Jeremy Boynes <jb...@apache.org>.
On Sep 17, 2013, at 11:10 PM, Violeta Georgieva <mi...@gmail.com> wrote:

> Hi Jeremy,
> 
> 2013/9/13 <bu...@apache.org>
>> 
>> https://issues.apache.org/bugzilla/show_bug.cgi?id=55508
>> 
>> Jeremy Boynes <jb...@apache.org> changed:
>> 
>>           What    |Removed                     |Added
>> 
> ----------------------------------------------------------------------------
>>             Status|RESOLVED                    |REOPENED
>>         Resolution|INVALID                     |---
>> 
>> --- Comment #4 from Jeremy Boynes <jb...@apache.org> ---
>> Reopening this for further discussion, and to see whether I need to change
>> TldScanner in trunk as I think I introduced a regression here.
>> 
>> I interpreted the spec as saying that all TLDs under /WEB-INF *must* be
>> included except for any under /WEB-INF/tags (except for any implicit.tld
> found
>> there). As a result, although applications *should* not place them in
>> /WEB-INF/lib or /WEB-INF/classes, that was a recommendation and if they
> were
>> there the container would need to include them.
>> 
>> The change to TldScanner in TC8 unintentionally resurrected the behaviour
> from
>> TC6 that was removed in TC7.
> 
> Do we have this change in TC7 or it is only in TC8?

It is only in TC8.
Cheers
Jeremy


Re: [Bug 55508] TldLocationsCache not recognising custom TLDs in WEB-INF/lib

Posted by Violeta Georgieva <mi...@gmail.com>.
Hi Jeremy,

2013/9/13 <bu...@apache.org>
>
> https://issues.apache.org/bugzilla/show_bug.cgi?id=55508
>
> Jeremy Boynes <jb...@apache.org> changed:
>
>            What    |Removed                     |Added
>
----------------------------------------------------------------------------
>              Status|RESOLVED                    |REOPENED
>          Resolution|INVALID                     |---
>
> --- Comment #4 from Jeremy Boynes <jb...@apache.org> ---
> Reopening this for further discussion, and to see whether I need to change
> TldScanner in trunk as I think I introduced a regression here.
>
> I interpreted the spec as saying that all TLDs under /WEB-INF *must* be
> included except for any under /WEB-INF/tags (except for any implicit.tld
found
> there). As a result, although applications *should* not place them in
> /WEB-INF/lib or /WEB-INF/classes, that was a recommendation and if they
were
> there the container would need to include them.
>
> The change to TldScanner in TC8 unintentionally resurrected the behaviour
from
> TC6 that was removed in TC7.

Do we have this change in TC7 or it is only in TC8?

Thanks
Violeta