DO NOT REPLY [Bug 42085] New: - java.util.Logging duplicated since 5.5.23. 5.5.20 is OK

[bugzilla]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=42085

   Summary: java.util.Logging duplicated since 5.5.23.  5.5.20 is OK
   Product: Tomcat 5
   Version: 5.5.23
  Platform: All
OS/Version: All
Status: NEW
  Severity: regression
  Priority: P3
 Component: Catalina:Modules
AssignedTo: tomcat-dev@jakarta.apache.org
ReportedBy: [EMAIL PROTECTED]


We use the followwing logging.properties in our WebApplication:

--%<%<
handlers = 1tivoli.java.util.logging.FileHandler,
2product.java.util.logging.FileHandler

# Handlers for the root logger
.handlers = 1tivoli.java.util.logging.FileHandler,
2product.java.util.logging.FileHandler

1tivoli.java.util.logging.FileHandler.level = INFO
1tivoli.java.util.logging.FileHandler.pattern = /var/log/product/tivoli%g.log
# limit Output to 1MB
1tivoli.java.util.logging.FileHandler.limit=1048576
# Limit rotation to 10 Logs ...
1tivoli.java.util.logging.FileHandler.count=10
1tivoli.java.util.logging.FileHandler.append=true
1tivoli.java.util.logging.FileHandler.formatter=de.customer.product.tivoli.TivoliFormatter

2product.java.util.logging.FileHandler.level = FINEST
2product.java.util.logging.FileHandler.pattern = /var/log/product/product%g.log
# limit Output to 1MB
2product.java.util.logging.FileHandler.limit=1048576
# Limit rotation to 5 Logs ...
2product.java.util.logging.FileHandler.count=5
2product.java.util.logging.FileHandler.append=true
2product.java.util.logging.FileHandler.formatter=java.util.logging.SimpleFormatter
--%<%<

When using apache-tomcat-5.5.23 every Line in the Logfiles
is duplicated. Debugging I found that the TivoliFormatter 
is actually created twice. And the root-logger has actually
4 Handlers and not two as intended. This does not allow us an
upgrade beyond 5.5.20.

Found on Linux64Bit and Windows-XP, both with JDK 1.5_R11

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

DO NOT REPLY [Bug 41430] - JkOptions +ForwardDirectories with Apache's DirectoryIndex

[bugzilla]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=41430





--- Additional Comments From [EMAIL PROTECTED]  2007-04-10 19:29 ---
This seems very similar to an issue I am having with JK 1.2.21.
a request for 
http://www.domain.tld/
returns the tomcat ROOT webapp index.jsp but I haven't a JkMount for the root
context.

in my httpd.conf, I have a 
DirectoryIndex /myapp/homepage.html
and
JkMount /myapp/* server1

The problem goes away when I revert back to 1.2.18 mod_jk.so.  Issue not in
1.2.19, introduced in 1.2.20

note: the local links in the ROOT webapp index.jsp do not work (as expected)
because that context is not mapped using modjk - yet that index.jsp is still 
served.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

svn commit: r527334 - /tomcat/tc6.0.x/tags/TOMCAT_6_0_11/

[remm]

Author: remm
Date: Tue Apr 10 15:56:49 2007
New Revision: 527334

URL: http://svn.apache.org/viewvc?view=rev&rev=527334
Log:
- 6.0.11. Candidate binaries will be available tomorrow.

Added:
tomcat/tc6.0.x/tags/TOMCAT_6_0_11/
  - copied from r527333, tomcat/tc6.0.x/trunk/


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

svn commit: r527331 - /tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

[remm]

Author: remm
Date: Tue Apr 10 15:49:53 2007
New Revision: 527331

URL: http://svn.apache.org/viewvc?view=rev&rev=527331
Log:
- Changelog update.

Modified:
tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

Modified: tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml?view=diff&rev=527331&r1=527330&r2=527331
==
--- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Tue Apr 10 15:49:53 2007
@@ -15,6 +15,14 @@
 
 
 
+  
+
+  
+Update DBCP to 1.2.2, pool to 1.3, JDT to 3.2.2 and remove collections
+build dependency (pero, remm)
+  
+
+  
   
 
   



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Re: Annotation processing - Geronimo injection

[David Jencks]

I've come up with a couple more patches that try to address your  
concerns:


https://issues.apache.org/jira/secure/attachment/12355259/ 
GERONIMO-3010-3c.patch
This one eliminates backwards compatibility support for  
AnnotationProcessor and eliminates the replaceable aspects of jasper  
InstanceManagerFactory, just leaving a static method for obtaining  
the InstanceFactory.  This is decidedly less flexible than the  
previous implementation and now requires jasper to be run in a  
container that supplies an InstanceManager.  There are still separate  
interfaces for InstanceManagers for catalina and jasper.  I think  
this is a considerably superior design than the next one...


This has 2 interfaces, 1 implementation (in tomcat, there's an unused  
impl in jasper, see below).



https://issues.apache.org/jira/secure/attachment/12355273/ 
GERONIMO-3010-4.patch
In addition, this one combines the InstanceManager interfaces.  I  
think this is a bad idea because it forces jasper to use an interface  
shared with catalina, which the previous patch does not.  This makes  
catalina marginally more complicated when used as a standalone  
container for jasper but makes it possible to use jasper outside  
catalina without any catalina classes.  Note also that the previous  
(3c) patch allows use of tomcat embedded in a container that supplies  
its own InstanceManager implementation without pulling in any jasper  
classes at runtime.


This has 1 interface, 1 implementation.

Both of these include a jasper DefaultInstanceManager that isn't  
used, but could be by a project that wants to install jasper with no  
annotation support.


I'd like to explore exactly how jasper creates classloaders and  
whether its possible to use class reloading features of modern jvms  
to eliminate the need to allow replacing the classloader in the  
jasper InstanceManager at runtime.  Unfortunately I doubt I'll have  
time for this in the next month or so.


On Apr 6, 2007, at 5:03 AM, Remy Maucherat wrote:


David Jencks wrote:

but I won't put it in the org.apache package.


There are not too many solutions to this problem, and  
(unfortunately for you, apparently) I think it's the most  
appropriate, or least inappropriate if you prefer. I was already  
aware it was possible to have one interface in Jasper and implement  
it in Catalina, or one interface in Catalina and use it in Jasper,  
and it would work ;)


yes, and I prefer 2 interfaces with the possiblity of supplying  
separate implementations.  This makes catalina's standalone life  
slightly more complicated but jasper's life outside catalina simpler  
IMO.


- I still don't know if I agree with removing the security checks  
out of Catalina, merging checks between filters and servlets, etc
I think that the security checks and related code are really  
specific to the tomcat classloader and you shouldn't force them on  
containers embedding tomcat that use different classloaders.  As  
far as uniformizing the code between servlets, filters, and  
listeners, I still don't understand the problem.  You might well  
be right, but I'd really appreciate concrete details on why some  
checks are appropriate for servlets and not listeners or filters.   
Since all of these objects are only created once (except for  
single-thread-model servlets, which have all the checks anyway) I  
don't really see why a slight difference in startup speed would  
outweigh the simplicity of uniform code and relative ease of  
understanding of having the object creation policy in one place.


I don't know for sure about that, I would have to look at what gets  
run in each case. Maybe the implementation in the proposed patch is  
fine.


I could have missed something, but so far I think what I'm proposing  
is OK.


- Class hierarchy for InstanceManager -> meh (although I suppose  
it perfectly fits your needs, though ...)
I agree... Geronimo implements its own InstanceManagers for tomcat  
and jasper, so we don't have that problem.  I'd be perfectly happy  
to drop support for AnnotationProcessor entirely, but I kinda  
doubt you would like it so much :-)


It does not make any difference, really: if the API changes  
significantly, there's no need to keep the old API and provide fake  
compatibility, it would not really help IMO.


I was complaining about the 2 interfaces + 1 abstract + 2 impls + 1  
factory (maybe I forgot some). This is overengineered (for Tomcat  
at least) IMO. Personally, the thing that I would like would be at  
most: 1 shared interface, 1 abstract, 1 impl. OTOH, I think I  
accept the idea of the InstanceManager interface.


Overall, thanks for the patch, it gives a very good basis for work  
(esp if you did test it).



To be honest, this whole stuff may be 6.next material. We'll see.

thanks!


The first decision is to see if the API change is going to be  
acceptable in 6.0.x.


Looking forward to your comments...



Rémy


thanks
david jencks



--

Re: Tagging 1.2.22

[Guenter Knauf]

Hi Rainer,
> - Fix whitespace in Netware make file
non-important, was more a test that commit works again; please drop from the 
list.

Guenter.



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Re: Tagging 1.2.22

[Rainer Jung]

Hi Mladen,

thanks for your comments, and I totally agree with everything :)

Regards,

Rainer

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Re: Tagging 1.2.22

[Mladen Turk]

Rainer Jung wrote:


1) Things to add to the changelog
=



I added few. Some of them are waste of resource IMHO ;)



I found the following changes, which are missing in the change log. Not
all of them need to go there, but maybe Mladen, Jean-Frederic, Guenter
and Henri can check, which ones they did and maybe add a line to the
change log:

- Fix html syntax in news docs pages
- jkrelease.sh: new users root:bin, automate file list for NEWS,
compress with gtar, don't set ANT_HOME and JAVA_HOME
- New build files on Win (nmake)
- Add tc 6 examples to config example and iis config example uriworkermap.
- apache: rename file_logger_t to jk_file_logger_t
- logging: refactor line endings logging to make it correct for all
platforms and webservers
- apache 1.3: rename child exit handler and add shm close
- apache 1.3: new child init handler with shm attach
- apache: add info level message during mod_jk startup with version info
- apache 2.2: win build env var APACHE22 und APACHE22_HOME and correct
apr/apr-1 distinction
- Fix whitespace in Netware make file
- Updates to iSeries Make file
- apache 2.0: use plog instead of pconf when initializing loggers
- Use fputs instead of fwrite in logging
- Translate charsets when dumping buffers and using EBCDIC
- Add worker names to some log messages in jk_ajp_common.c

2) Missing subversion properties corrected
==

I added subversion properties

svn:eol-style native
svn:keywords 'Author Date Id Revision'

to the following win files, which had no props at all:

jk/native/iis/pcre/pcre.amd64
jk/native/iis/pcre/pcre.ia64
jk/native/iis/pcre/dftables.x86
jk/native/iis/pcre/pcre.x86
jk/native/iis/Makefile.ia64
jk/native/iis/Makefile.x86
jk/native/iis/Makefile.amd64



Should have eol:native, thanks.



No all line endings should be platform native. I hope that's correct.



Sure, like all textual contents.


The other files with no properties were all binaries (*.exe, *.gif,
*.ico, *.rtf). More precisely, I fixed a couple of non-IIS files, and
now the remaining ones are only binary formats.



Perfect.


3) fail_on_status
=



Made that multi line.



4) load_properties/load_property




If that makes you happy, I'm sure they can be merged ;)



5) Reference links on NEWS file
===

The NEWS file contains links of the form in the reference section

file:///home/mturk/wrkplace/tomcat-connectors-1.1.22-dev-524775-src/



This is wrong links usage. Think it has the no reference option.
I'll investigate and update jkrelease.sh



7) Missing docs
===

Mladen, maybe you can add a few lines to the two IIS docs pages about
the new cool URL rewriting? Or do you want to release the new feature in
stealth mode and document in 1.2.23?



Have no time for documentation right now.
Nevertheless, it can be added later. There is no need to
be the part of the package itself.
IMHO the one that counts is on the tomcat.apache.org site.



9) JK_SHM_LOCK_REOPEN
=

What's the use of JK_SHM_LOCK_REOPEN?



Something no one will ever use I hope ;)
On some kernels the flock was failing if inherited
from parent, but that is kernel bug, not ours.
To fight with that we by default use fcntl, that
BTW allows to have the locks on NFS shares as well.
For those that wish tho experiment there is
configure --enable-flock if you know your locks won't be on
the NFS share and don't have buggy kernel.

In general JK_SHM_REOPEN was added to prove the kernel bug,
but since it's a hack and security leach it'll go away.
IMHO it is of developers interest only.

Regards,
Mladen.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Re: Jk_isapi_plugin.c bug #42038 in JK 1.2.20, 21, 22

[Mladen Turk]

Coffman, Steven wrote:

Hi,
I have a question for 42038. I only recently figured out exactly what
was causing it, so I'm sorry for the bugzilla spam as I zeroed in on it.



I already explain that in the bz case.
Please do not steal threads!

Regards,
Mladen.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

svn commit: r527143 - in /tomcat/connectors/trunk/jk: native/common/jk_util.c xdocs/miscellaneous/changelog.xml

[mturk]

Author: mturk
Date: Tue Apr 10 07:29:48 2007
New Revision: 527143

URL: http://svn.apache.org/viewvc?view=rev&rev=527143
Log:
Make fail_on_status multiline

Modified:
tomcat/connectors/trunk/jk/native/common/jk_util.c
tomcat/connectors/trunk/jk/xdocs/miscellaneous/changelog.xml

Modified: tomcat/connectors/trunk/jk/native/common/jk_util.c
URL: 
http://svn.apache.org/viewvc/tomcat/connectors/trunk/jk/native/common/jk_util.c?view=diff&rev=527143&r1=527142&r2=527143
==
--- tomcat/connectors/trunk/jk/native/common/jk_util.c (original)
+++ tomcat/connectors/trunk/jk/native/common/jk_util.c Tue Apr 10 07:29:48 2007
@@ -142,6 +142,7 @@
 USER_OF_WORKER,
 GOOD_RATING_OF_WORKER,
 BAD_RATING_OF_WORKER,
+STATUS_FAIL_OF_WORKER,
 "list",
 NULL
 };
@@ -188,7 +189,6 @@
 XML_NAMESPACE_OF_WORKER,
 XML_DOCTYPE_OF_WORKER,
 PROP_PREFIX_OF_WORKER,
-STATUS_FAIL_OF_WORKER,
 USER_CASE_OF_WORKER,
 NULL
 };

Modified: tomcat/connectors/trunk/jk/xdocs/miscellaneous/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/connectors/trunk/jk/xdocs/miscellaneous/changelog.xml?view=diff&rev=527143&r1=527142&r2=527143
==
--- tomcat/connectors/trunk/jk/xdocs/miscellaneous/changelog.xml (original)
+++ tomcat/connectors/trunk/jk/xdocs/miscellaneous/changelog.xml Tue Apr 10 
07:29:48 2007
@@ -27,6 +27,16 @@
   
 
   
+  Refactor line endings logging to make it correct for all
+  platforms and webservers. (mturk)
+  
+  
+  Added command line windows make files. (mturk)
+  
+  
+  Allow fail_on_status directive to be multi line. (mturk)
+  
+  
 42076: Fix name of new option from ForwardCertChain to
 ForwardSSLCertChain as documented. (rjung)
   
@@ -479,7 +489,7 @@
   LB: Cleanup of Mladens forced recovery. (rjung)
   
   
-  LB: Do not change lb_value for recovering workers to max, if 
+  LB: Do not change lb_value for recovering workers to max, if
   we are using BUSYNESS method. (rjung)
   
   



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Jk_isapi_plugin.c bug #42038 in JK 1.2.20, 21, 22

[Coffman, Steven]

Hi,
I have a question for 42038. I only recently figured out exactly what
was causing it, so I'm sorry for the bugzilla spam as I zeroed in on it.

(FYI, On IIS, virtual host name can be optionally prefixed, so /*
positive mount always succeeds even if there are non-virtual exclusion
mounts) 

Even if I reverse the order in jk_isapi_plugin.c so that it checks
Default redirection map_uri_to_worker before attempting a Virtual Host
Redirection map_uri_to_worker, it then still needs additional
information to be communicated (that it both matched and no matched for
Default so it should skip Virtual). That either means an additional
parameter (with side effects) passed to map_uri_to_worker, or a global
variable. Either way, it means changing code other code that uses the
map_uri_to_worker. I would prefer to add another parameter.

Is this legit? Can I submit a patch? Or is there an alternate approach
that is preferred?

PS. I just now found Rainer's message:
[snip]
8) BZ http://issues.apache.org/bugzilla/show_bug.cgi?id=42038
=

This is special to IIS. I think it's not a show stopper (it's also not
new), but interesting. Because IIS has the option of prefixing the URL
with the vhost name, exclusion mounts do not work, once you add /* the
the list of positive mounts.

[snip]

Again, sorry for the bugzilla spam.
-Steve

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

DO NOT REPLY [Bug 42038] - JK uriworkermap ROOT mapping causes non-virtual exclusions to be ignored

[bugzilla]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=42038


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||WONTFIX




--- Additional Comments From [EMAIL PROTECTED]  2007-04-10 07:08 ---
Hi,

> /*=v7CDALiveWorker
This is wrong.
In case you have multiple virtual hosts you shuld never do that.
Always use the /VHOST/URI form:
/lsa-cmsdev1.lsa.adsroot.itcs.umich.edu/*=v7CDALiveWorker
or at least ...
/*/*=v7CDALiveWorker

> #Exclude static files (take turns commenting out the following for testing)
> !/lsa-cmsdev1.lsa.adsroot.itcs.umich.edu/UMICH/*=v7CDALiveWorker
> !/UMICH/*=v7CDALiveWorker

Again, use the:
!/*/UMICH/*=v7CDALiveWorker

There is simply no way to know what the virtual host is from the
uriworkermap.properties unless we change the configuration, and
mark the VHOST part somehow. That would make configs backward incompatible,
so it'll have to wait some post 1.2 version


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

DO NOT REPLY [Bug 42076] - typo in ForwardSSLCertChain JkOptions

[bugzilla]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=42076


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2007-04-10 03:06 ---
This fix will be part of 1.2.22, which is the first release with this new 
feature.
The release will be done soon.
Thanks for reporting this in time.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

svn commit: r527079 - /tomcat/connectors/trunk/jk/xdocs/miscellaneous/changelog.xml

[rjung]

Author: rjung
Date: Tue Apr 10 03:05:16 2007
New Revision: 527079

URL: http://svn.apache.org/viewvc?view=rev&rev=527079
Log:
Add changelog item for fixing ForwardCertChain -> ForwardSSLCertChain.
(BZ 42076)

Modified:
tomcat/connectors/trunk/jk/xdocs/miscellaneous/changelog.xml

Modified: tomcat/connectors/trunk/jk/xdocs/miscellaneous/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/connectors/trunk/jk/xdocs/miscellaneous/changelog.xml?view=diff&rev=527079&r1=527078&r2=527079
==
--- tomcat/connectors/trunk/jk/xdocs/miscellaneous/changelog.xml (original)
+++ tomcat/connectors/trunk/jk/xdocs/miscellaneous/changelog.xml Tue Apr 10 
03:05:16 2007
@@ -27,6 +27,10 @@
   
 
   
+42076: Fix name of new option from ForwardCertChain to
+ForwardSSLCertChain as documented. (rjung)
+  
+  
   Docs: Fix a couple of typos, change format of a few tables,
   fix links to news pages. (rjung)
   



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

DO NOT REPLY [Bug 42077] New: - the iterator from javax.el.CompositeELResolver#getFeatureDescriptor() must not return null values

[bugzilla]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=42077

   Summary: the iterator from
javax.el.CompositeELResolver#getFeatureDescriptor() must
not return null values
   Product: Tomcat 6
   Version: 6.0.11
  Platform: All
OS/Version: All
Status: NEW
  Severity: major
  Priority: P2
 Component: Catalina
AssignedTo: tomcat-dev@jakarta.apache.org
ReportedBy: [EMAIL PROTECTED]


According to the spec the iterator which is returned by
javax.el.CompositeELResolver#getFeatureDescriptor() must skip any null
FeatureDescriptor of a containing el resolver.

Currently any feature descriptor value is returned whether it is null or not.

I have a working testcase (including a fix) in myfaces 1.2 
TestCase:
http://svn.apache.org/viewvc/myfaces/core/branches/jsf12/impl/src/test/java/org/apache/myfaces/el/CompositeELResolverTest.java?revision=527076&pathrev=527076
Fix:
http://svn.apache.org/viewvc/myfaces/core/branches/jsf12/impl/src/main/java/org/apache/myfaces/el/CompositeELResolver.java?revision=527076&pathrev=527076

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

svn commit: r527077 - in /tomcat/connectors/trunk/jk/native: apache-1.3/mod_jk.c apache-2.0/mod_jk.c

[rjung]

Author: rjung
Date: Tue Apr 10 03:03:05 2007
New Revision: 527077

URL: http://svn.apache.org/viewvc?view=rev&rev=527077
Log:
Fix new option ForwardCertChain -> ForwardSSLCertChain
like it was documented.

Modified:
tomcat/connectors/trunk/jk/native/apache-1.3/mod_jk.c
tomcat/connectors/trunk/jk/native/apache-2.0/mod_jk.c

Modified: tomcat/connectors/trunk/jk/native/apache-1.3/mod_jk.c
URL: 
http://svn.apache.org/viewvc/tomcat/connectors/trunk/jk/native/apache-1.3/mod_jk.c?view=diff&rev=527077&r1=527076&r2=527077
==
--- tomcat/connectors/trunk/jk/native/apache-1.3/mod_jk.c (original)
+++ tomcat/connectors/trunk/jk/native/apache-1.3/mod_jk.c Tue Apr 10 03:03:05 
2007
@@ -1738,7 +1738,7 @@
 else if (!strcasecmp(w, "DisableReuse")) {
 opt = JK_OPT_DISABLEREUSE;
 }
-else if (!strcasecmp(w, "ForwardCertChain")) {
+else if (!strcasecmp(w, "ForwardSSLCertChain")) {
 opt = JK_OPT_FWDCERTCHAIN;
 }
 else

Modified: tomcat/connectors/trunk/jk/native/apache-2.0/mod_jk.c
URL: 
http://svn.apache.org/viewvc/tomcat/connectors/trunk/jk/native/apache-2.0/mod_jk.c?view=diff&rev=527077&r1=527076&r2=527077
==
--- tomcat/connectors/trunk/jk/native/apache-2.0/mod_jk.c (original)
+++ tomcat/connectors/trunk/jk/native/apache-2.0/mod_jk.c Tue Apr 10 03:03:05 
2007
@@ -1771,7 +1771,7 @@
 else if (!strcasecmp(w, "DisableReuse")) {
 opt = JK_OPT_DISABLEREUSE;
 }
-else if (!strcasecmp(w, "ForwardCertChain")) {
+else if (!strcasecmp(w, "ForwardSSLCertChain")) {
 opt = JK_OPT_FWDCERTCHAIN;
 }
 else



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

DO NOT REPLY [Bug 42076] New: - typo in ForwardSSLCertChain JkOptions

[bugzilla]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=42076

   Summary: typo in  ForwardSSLCertChain JkOptions
   Product: Tomcat 5
   Version: Unknown
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: P2
 Component: Native:JK
AssignedTo: tomcat-dev@jakarta.apache.org
ReportedBy: [EMAIL PROTECTED]


Recently, ForwardSSLCertChain option was added to mod_jk for apache.

Although document and sources' comment indicate ForwardSSLCertChain, but code is
written as ForwardCertChain.

Here is a patch for fixing typo

Index: apache-1.3/mod_jk.c
===
--- apache-1.3/mod_jk.c (revision 527056)
+++ apache-1.3/mod_jk.c (working copy)
@@ -1738,7 +1738,7 @@
 else if (!strcasecmp(w, "DisableReuse")) {
 opt = JK_OPT_DISABLEREUSE;
 }
-else if (!strcasecmp(w, "ForwardCertChain")) {
+else if (!strcasecmp(w, "ForwardSSLCertChain")) {
 opt = JK_OPT_FWDCERTCHAIN;
 }
 else
Index: apache-2.0/mod_jk.c
===
--- apache-2.0/mod_jk.c (revision 527056)
+++ apache-2.0/mod_jk.c (working copy)
@@ -1771,7 +1771,7 @@
 else if (!strcasecmp(w, "DisableReuse")) {
 opt = JK_OPT_DISABLEREUSE;
 }
-else if (!strcasecmp(w, "ForwardCertChain")) {
+else if (!strcasecmp(w, "ForwardSSLCertChain")) {
 opt = JK_OPT_FWDCERTCHAIN;
 }
 else

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]