harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sian January (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HARMONY-617) [classlib][luni] HttpURLConnection does not support persistent connections
Date Fri, 09 Feb 2007 14:21:05 GMT

    [ https://issues.apache.org/jira/browse/HARMONY-617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12471704

Sian January commented on HARMONY-617:

Just a couple more comments.  I don't think we need to do SecurityManager checkConnect because
we use Sockets under the covers and checkConnect is always called on creation of a new Socket.
 Surely we don't need to call it more than once for each connection?  

Also I think it would be good if we could test connections to the internet, because if we
don't do that we're just testing that the client and server match up if you see what I mean,
and we're not testing the code with a real http server.  However if you think this is bad
practice or we're not allowed to do it then the tests that use an internet connection could
probably be re-written to use the new Mock HTTP Server you wrote.

> [classlib][luni] HttpURLConnection does not support persistent connections
> --------------------------------------------------------------------------
>                 Key: HARMONY-617
>                 URL: https://issues.apache.org/jira/browse/HARMONY-617
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>            Reporter: Alexander Kleymenov
>         Assigned To: Paulex Yang
>         Attachments: luni.patch, test.patch
> Current HttpURLConnection does not support persistent connections while RI implementation
> Implementation of this mechanism will allow sufficiently increase the performance of
network applications based on HttpURLConnections. To make Harmony's implementation more attractive
for use this support SHALL be implemented.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message