[ 
http://issues.apache.org/jira/browse/HTTPCORE-14?page=comments#action_12438493 
] 
            
Roland Weber commented on HTTPCORE-14:
--------------------------------------

Hi Oleg,

I think the scheme should remain in HttpHost as a String. Maybe with automatic 
lowercase conversion.
The getter method could be called getSchemeName instead of getScheme, to avoid 
confusion.
Otherwise, I'm fine with this change.

(I have one more idea for HttpCore, but it's rather a side issue and doesn't 
have to go into the next alpha.)

cheers,
  Roland


> Consider decoupling HttpHost and Scheme and moving Scheme, SocketFactory and 
> SecureSocketFactory to HttpConn
> ------------------------------------------------------------------------------------------------------------
>
>                 Key: HTTPCORE-14
>                 URL: http://issues.apache.org/jira/browse/HTTPCORE-14
>             Project: HttpComponents Core
>          Issue Type: Improvement
>          Components: HttpCore
>            Reporter: Oleg Kalnichevski
>            Priority: Minor
>         Attachments: 20060927-httphost.patch
>
>
> SocketFactory interface in HttpCore implies a blocking process of opening a 
> new socket. Moreover, there is no way to interrupt the process of opening a 
> connection due to limitations of Java 1.3 API. 
> I suggest the SocketFactory and SecureSocketFactory interfaces and the Scheme 
> class dependent on those interfaces be moved to HttpConn and the HttpHost 
> class be decoupled from Scheme.
> (This is going to be the last controversial bit I wanted to propose and once 
> this issue is resolved I can safely declare I am done with HttpCore)
> Oleg

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to