Re: RFR: JDK-8243376: java.net.SocketPermission.implies(Permission p) spec is mismatching with implementation [v9]

2021-12-20 Thread Jayashree S Kumar
On Wed, 10 Mar 2021 17:32:30 GMT, Jayashree S Kumar wrote: >> Issue >> >> https://bugs.openjdk.java.net/browse/JDK-8243376 >> >> Problem >> >> The scenario is: >> - Some specified target hostname resolves to two IP addresses (always the >

Re: RFR: JDK-8243376: java.net.SocketPermission.implies(Permission p) spec is mismatching with implementation [v9]

2021-03-10 Thread Jayashree S Kumar
implied SocketPermission hostname is > resolved, the resolved addresses order differs, and the internally saved IP > address mismatches, resulting on SocketPermission#implies() false. > > > Michael McMahon kindly reviewed and suggested changes: > https://mail.openjdk.java.net/pip

Re: RFR: JDK-8243376: java.net.SocketPermission.implies(Permission p) spec is mismatching with implementation [v6]

2021-03-10 Thread Jayashree S Kumar
On Fri, 5 Mar 2021 15:08:03 GMT, Michael McMahon wrote: >> Jayashree S Kumar has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Code Review: Incorporated Indention correction, added getIP and updated >> try

Re: RFR: JDK-8243376: java.net.SocketPermission.implies(Permission p) spec is mismatching with implementation [v8]

2021-03-10 Thread Jayashree S Kumar
implied SocketPermission hostname is > resolved, the resolved addresses order differs, and the internally saved IP > address mismatches, resulting on SocketPermission#implies() false. > > > Michael McMahon kindly reviewed and suggested changes: > https://mail.openjdk.java.net/pip

Re: RFR: JDK-8243376: java.net.SocketPermission.implies(Permission p) spec is mismatching with implementation [v7]

2021-03-10 Thread Jayashree S Kumar
implied SocketPermission hostname is > resolved, the resolved addresses order differs, and the internally saved IP > address mismatches, resulting on SocketPermission#implies() false. > > > Michael McMahon kindly reviewed and suggested changes: > https://mail.openjdk.java.net/pip

Re: RFR: JDK-8243376: java.net.SocketPermission.implies(Permission p) spec is mismatching with implementation [v6]

2021-03-10 Thread Jayashree S Kumar
On Fri, 5 Mar 2021 15:47:43 GMT, Vyom Tewari wrote: >> Changes requested by michaelm (Reviewer). > > As you change equal & hashCode method, if possible can you please add some > additional tests in "jdk/java/net/SocketPermission/Equals.java" just to make > sure we test every corner case. I

Re: RFR: JDK-8243376: java.net.SocketPermission.implies(Permission p) spec is mismatching with implementation [v6]

2021-02-09 Thread Jayashree S Kumar
implied SocketPermission hostname is > resolved, the resolved addresses order differs, and the internally saved IP > address mismatches, resulting on SocketPermission#implies() false. > > > Michael McMahon kindly reviewed and suggested changes: > https://mail.openjdk.java.net/pip

Re: RFR: JDK-8243376: java.net.SocketPermission.implies(Permission p) spec is mismatching with implementation [v4]

2021-02-09 Thread Jayashree S Kumar
On Mon, 8 Feb 2021 14:43:10 GMT, Michael McMahon wrote: >> Jayashree S Kumar has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Code Review: cname made array accounting for multiple cname values > > Changes r

Re: RFR: JDK-8243376: java.net.SocketPermission.implies(Permission p) spec is mismatching with implementation [v5]

2021-02-09 Thread Jayashree S Kumar
implied SocketPermission hostname is > resolved, the resolved addresses order differs, and the internally saved IP > address mismatches, resulting on SocketPermission#implies() false. > > > Michael McMahon kindly reviewed and suggested changes: > https://mail.openjdk.java.net/pip

Re: RFR: JDK-8243376: java.net.SocketPermission.implies(Permission p) spec is mismatching with implementation [v4]

2021-02-05 Thread Jayashree S Kumar
implied SocketPermission hostname is > resolved, the resolved addresses order differs, and the internally saved IP > address mismatches, resulting on SocketPermission#implies() false. > > > Michael McMahon kindly reviewed and suggested changes: > https://mail.openjdk.java.net/pip

Re: RFR: JDK-8243376: java.net.SocketPermission.implies(Permission p) spec is mismatching with implementation [v2]

2021-02-05 Thread Jayashree S Kumar
On Mon, 11 Jan 2021 15:04:03 GMT, Michael McMahon wrote: >> Jayashree S Kumar has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Incorporated changes suggested by Vyom in testcase > > Hi Jay, > Looking bac

Re: RFR: JDK-8243376: java.net.SocketPermission.implies(Permission p) spec is mismatching with implementation [v3]

2021-02-05 Thread Jayashree S Kumar
implied SocketPermission hostname is > resolved, the resolved addresses order differs, and the internally saved IP > address mismatches, resulting on SocketPermission#implies() false. > > > Michael McMahon kindly reviewed and suggested changes: > https://mail.openjdk.java.net/pip

Re: RFR: JDK-8243376: java.net.SocketPermission.implies(Permission p) spec is mismatching with implementation [v2]

2021-01-05 Thread Jayashree S Kumar
implied SocketPermission hostname is > resolved, the resolved addresses order differs, and the internally saved IP > address mismatches, resulting on SocketPermission#implies() false. > > > Michael McMahon kindly reviewed and suggested changes: > https://mail.openjdk.java.net/pip

Re: RFR: JDK-8243376: java.net.SocketPermission.implies(Permission p) spec is mismatching with implementation

2021-01-05 Thread Jayashree S Kumar
On Sat, 2 Jan 2021 10:11:30 GMT, Jayashree S Kumar wrote: > Issue > > https://bugs.openjdk.java.net/browse/JDK-8243376 > > Problem > > The scenario is: > - Some specified target hostname resolves to two IP addresses (always the > same address pair). > - The

RFR: JDK-8243376: java.net.SocketPermission.implies(Permission p) spec is mismatching with implementation

2021-01-05 Thread Jayashree S Kumar
Issue https://bugs.openjdk.java.net/browse/JDK-8243376 Problem The scenario is: - Some specified target hostname resolves to two IP addresses (always the same address pair). - The DNS resolved order of the two ip addresses changes (a usual LoadBalancer type behavior). - The CNAME of the