[Bug 63550] LDAP non standard port leads to JNDIRealm erratic behaviour

2019-07-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=63550 --- Comment #6 from Michael Osipov --- My proposal works if you use JNDI with InitialDirContext directly. I don't use the JNDIRealm, I use a custom ActiveDirectoryRealm which is perfectly coded for AD only. Our connetion URL contains either mul

[Bug 63550] LDAP non standard port leads to JNDIRealm erratic behaviour

2019-07-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=63550 --- Comment #5 from Eugène Adell --- (In reply to Michael Osipov from comment #4) > That's the wrong spot, you need to look here: > https://docs.oracle.com/javase/7/docs/technotes/guides/jndi/jndi-ldap-gl. > html#url Thank you Michael, that do

[GitHub] [tomcat] govi20 edited a comment on issue #177: Code polish: replace classic for-loop with enhanced

2019-07-07 Thread GitBox
govi20 edited a comment on issue #177: Code polish: replace classic for-loop with enhanced URL: https://github.com/apache/tomcat/pull/177#issuecomment-509019980 Yes. `for-loop's` temp variable and `ServletContextAttributeListener's` old var name were clashing. I don't think using `tempList

[GitHub] [tomcat] govi20 edited a comment on issue #177: Code polish: replace classic for-loop with enhanced

2019-07-07 Thread GitBox
govi20 edited a comment on issue #177: Code polish: replace classic for-loop with enhanced URL: https://github.com/apache/tomcat/pull/177#issuecomment-509019980 Yes. `for-loop's` temp variable and `ServletContextAttributeListener's` old var name were clashing. I don't think using `tempList

[GitHub] [tomcat] govi20 commented on issue #177: Code polish: replace classic for-loop with enhanced

2019-07-07 Thread GitBox
govi20 commented on issue #177: Code polish: replace classic for-loop with enhanced URL: https://github.com/apache/tomcat/pull/177#issuecomment-509019980 Yes. `for-loop's` temp and `ServletContextAttributeListener's` old name are clashing. I don't think using `tempListener` in a `for-loop`

[GitHub] [tomcat] ChristopherSchultz commented on issue #177: Code polish: replace classic for-loop with enhanced

2019-07-07 Thread GitBox
ChristopherSchultz commented on issue #177: Code polish: replace classic for-loop with enhanced URL: https://github.com/apache/tomcat/pull/177#issuecomment-509018351 The patch is smaller is you don't replace `listener` with `servletContextAttributeListener` everywhere. Any reason to re-nam

[Bug 55662] Add a way to set an instance of java.sql.Driver directly on org.apache.tomcat.jdbc.pool.DataSource

2019-07-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=55662 --- Comment #3 from Michael --- I had the same problems with SQL on this web page https://detecthistory.com/best-metal-detectors/ -- You are receiving this mail because: You are the assignee for the bug. --

[Bug 63550] LDAP non standard port leads to JNDIRealm erratic behaviour

2019-07-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=63550 --- Comment #4 from Michael Osipov --- (In reply to Eugène Adell from comment #3) > (In reply to Michael Osipov from comment #2) > > Don't use the altenative URL. Oracle's default implementation for LDAP will > > handle multiple space-separated

[GitHub] [tomcat] govi20 opened a new pull request #177: Code polish: replace classic for-loop with enhanced

2019-07-07 Thread GitBox
govi20 opened a new pull request #177: Code polish: replace classic for-loop with enhanced URL: https://github.com/apache/tomcat/pull/177 ApplicationContext has classic-for loops, these for loops can be replaced with enhanced for-loop to improve code readability. -

[Bug 63550] LDAP non standard port leads to JNDIRealm erratic behaviour

2019-07-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=63550 --- Comment #3 from Eugène Adell --- (In reply to Michael Osipov from comment #2) > Don't use the altenative URL. Oracle's default implementation for LDAP will > handle multiple space-separated URLs properly. Is it documented anywhere ? I don'

[Bug 63550] LDAP non standard port leads to JNDIRealm erratic behaviour

2019-07-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=63550 --- Comment #2 from Michael Osipov --- (In reply to Eugène Adell from comment #1) > JNDIRealm.java : > > if (connectionURL != null && connectionAttempt == 0) > env.put(Context.PROVIDER_URL, connectionURL); > else if

[Bug 63550] LDAP non standard port leads to JNDIRealm erratic behaviour

2019-07-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=63550 --- Comment #1 from Eugène Adell --- JNDIRealm.java : if (connectionURL != null && connectionAttempt == 0) env.put(Context.PROVIDER_URL, connectionURL); else if (alternateURL != null && connectionAttempt > 0)

[Bug 63550] New: LDAP non standard port leads to JNDIRealm erratic behaviour

2019-07-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=63550 Bug ID: 63550 Summary: LDAP non standard port leads to JNDIRealm erratic behaviour Product: Tomcat 9 Version: 9.0.x Hardware: All OS: All Stat

Bug report for Taglibs [2019/07/07]

2019-07-07 Thread bugzilla
+---+ | Bugzilla Bug ID | | +-+ | | Status: UNC=Unconfirmed NEW=New ASS=Assigned

Bug report for Tomcat Native [2019/07/07]

2019-07-07 Thread bugzilla
+---+ | Bugzilla Bug ID | | +-+ | | Status: UNC=Unconfirmed NEW=New ASS=Assigned

Bug report for Tomcat Modules [2019/07/07]

2019-07-07 Thread bugzilla
+---+ | Bugzilla Bug ID | | +-+ | | Status: UNC=Unconfirmed NEW=New ASS=Assigned

Bug report for Tomcat Connectors [2019/07/07]

2019-07-07 Thread bugzilla
+---+ | Bugzilla Bug ID | | +-+ | | Status: UNC=Unconfirmed NEW=New ASS=Assigned

Bug report for Tomcat 8 [2019/07/07]

2019-07-07 Thread bugzilla
+---+ | Bugzilla Bug ID | | +-+ | | Status: UNC=Unconfirmed NEW=New ASS=Assigned

Bug report for Tomcat 7 [2019/07/07]

2019-07-07 Thread bugzilla
+---+ | Bugzilla Bug ID | | +-+ | | Status: UNC=Unconfirmed NEW=New ASS=Assigned

Bug report for Tomcat 9 [2019/07/07]

2019-07-07 Thread bugzilla
+---+ | Bugzilla Bug ID | | +-+ | | Status: UNC=Unconfirmed NEW=New ASS=Assigned