RFR(s): 8228580: DnsClient TCP socket timeout
Milan Mimica
milan.mimica at gmail.com
Wed Sep 18 13:25:49 UTC 2019
Hi Pavel
Sure. Here is the incremental change:
http://cr.openjdk.java.net/~mmimica/8228580/webrev.03/
What actually bothers me from the beginning is the truncated response.
The TXT attribute, a String, prints "A very popular h", but does not
equals("A very popular h"), because of some stray bytes. I guess it's
because of how DNS response parsing works. I can imagine how this
could cause problems to users. I think, at least, we should have a way
to tell the user that the response is truncated, and the payload is
partial/invalid.
On Tue, 17 Sep 2019 at 15:09, Pavel Rappo <pavel.rappo at oracle.com> wrote:
>
> Milan,
>
> While the CSR is being processed, could we maybe think of some additional testing for that change? Otherwise, that test seems kind of anemic. It makes sure that the query doesn't hang, but that's about it. It doesn't check that the timeout is respected. I was wondering if you could propose some way of testing that.
>
> > On 17 Sep 2019, at 09:55, Pavel Rappo <pavel.rappo at oracle.com> wrote:
> >
> > I have filed the CSR:
> >
> > https://bugs.openjdk.java.net/browse/JDK-8230965
> >
> >> On 13 Sep 2019, at 11:21, Pavel Rappo <pavel.rappo at oracle.com> wrote:
> >>
> >> Here's the latest webrev accumulating all the changes we've discussed so far:
> >>
> >> http://cr.openjdk.java.net/~prappo/8228580/webrev.03/
> >>
> >> If people are okay with that I will proceed to creating a CSR.
>
--
Milan Mimica
More information about the core-libs-dev
mailing list