Integrated: 8288895: LdapContext doesn't honor set referrals limit

2022-06-30 Thread rmartinc
On Thu, 23 Jun 2022 08:20:31 GMT, rmartinc wrote: > Fixes [JDK-8288895](https://bugs.openjdk.org/browse/JDK-8288895). > > Any `LimitExceededException` now quits the referral loop. > > Added class `ReferralLimitSearchTest`. It is a simple jtreg test which

Re: RFR: 8288895: LdapContext doesn't honor set referrals limit [v3]

2022-06-29 Thread rmartinc
On Tue, 28 Jun 2022 14:56:30 GMT, Aleksei Efimov wrote: >> rmartinc has updated the pull request incrementally with one additional >> commit since the last revision: >> >> 8288895: LdapContext doesn't honor set referrals limit (typo) > > Marked as revie

Re: RFR: 8288895: LdapContext doesn't honor set referrals limit [v3]

2022-06-28 Thread rmartinc
ed. The dummy ldap > server always returns a referral for any search query. If the number of > searches exceeds the limit, it throws an `IOException` and the test fails. > > > make test TEST=jtreg:jdk/com/sun/jndi/ldap/ReferralLimitSearchTest.java rmartinc has updated the pull request

Re: RFR: 8288895: LdapContext doesn't honor set referrals limit [v2]

2022-06-28 Thread rmartinc
On Tue, 28 Jun 2022 11:21:44 GMT, Aleksei Efimov wrote: >> rmartinc has refreshed the contents of this pull request, and previous >> commits have been removed. The incremental views will show differences >> compared to the previous content of the PR. The pull request co

Re: RFR: 8288895: LdapContext doesn't honor set referrals limit [v2]

2022-06-28 Thread rmartinc
On Tue, 28 Jun 2022 07:07:24 GMT, rmartinc wrote: >> src/java.naming/share/classes/com/sun/jndi/ldap/AbstractLdapNamingEnumeration.java >> line 339: >> >>> 337: break; >>> 338: } else if (errEx == null) { &g

Re: RFR: 8288895: LdapContext doesn't honor set referrals limit [v2]

2022-06-28 Thread rmartinc
On Mon, 27 Jun 2022 14:57:30 GMT, Daniel Fuchs wrote: >> rmartinc has refreshed the contents of this pull request, and previous >> commits have been removed. The incremental views will show differences >> compared to the previous content of the PR. The pull request contains

Re: RFR: 8288895: LdapContext doesn't honor set referrals limit [v2]

2022-06-28 Thread rmartinc
ed. The dummy ldap > server always returns a referral for any search query. If the number of > searches exceeds the limit, it throws an `IOException` and the test fails. > > > make test TEST=jtreg:jdk/com/sun/jndi/ldap/ReferralLimitSearchTest.java rmartinc has refreshed the contents o

RFR: 8288895: LdapContext doesn't honor set referrals limit

2022-06-24 Thread rmartinc
Fixes [JDK-8288895](https://bugs.openjdk.org/browse/JDK-8288895). Any `LimitExceededException` now quits the referral loop. Added class `ReferralLimitSearchTest`. It is a simple jtreg test which checks that `java.naming.ldap.referral.limit` is really enforced. The dummy ldap server always retur