[jira] Created: (HTTPCLIENT-824) Review the use of BaiscHttpParams and HttpProtocolProcessor in HttpClient

View: New views
3 Messages — Rating Filter:   Alert me  

[jira] Created: (HTTPCLIENT-824) Review the use of BaiscHttpParams and HttpProtocolProcessor in HttpClient

by JIRA jira@apache.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Review the use of BaiscHttpParams and HttpProtocolProcessor in HttpClient
-------------------------------------------------------------------------

                 Key: HTTPCLIENT-824
                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-824
             Project: HttpComponents HttpClient
          Issue Type: Task
          Components: HttpClient
            Reporter: Oleg Kalnichevski
             Fix For: 4.0 Beta 3


Review the use of BaiscHttpParams and HttpProtocolProcessor in HttpClient and replace with thread-safe implementations where necessary.

Oleg

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


[jira] Updated: (HTTPCLIENT-824) Review the use of BaiscHttpParams and HttpProtocolProcessor in HttpClient

by JIRA jira@apache.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message


     [ https://issues.apache.org/jira/browse/HTTPCLIENT-824?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Oleg Kalnichevski updated HTTPCLIENT-824:
-----------------------------------------

    Fix Version/s:     (was: 4.0 Beta 3)
                   4.1.0

I reviewed the code and found the following:

(1) HttpClient correctly synchronizes access to its ProtocolProcessor instance variable. It also makes a copy of it before passing it the request director, so the master copy can be safely mutated while current requests are being executed. No problem here.

(2) Things are a bit messy with HttpParams. BasicHttpParams classes is used all over the place. Decorating those with a non-public thread-safe wrapper inside HttpClient does not really help as the user can still mutate its state using the original reference. The problem can be solved only by introducing a public thread-safe implementation of HttpParams, which we cannot do until 4.1

Oleg

> Review the use of BaiscHttpParams and HttpProtocolProcessor in HttpClient
> -------------------------------------------------------------------------
>
>                 Key: HTTPCLIENT-824
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-824
>             Project: HttpComponents HttpClient
>          Issue Type: Task
>          Components: HttpClient
>            Reporter: Oleg Kalnichevski
>             Fix For: 4.1.0
>
>
> Review the use of BaiscHttpParams and HttpProtocolProcessor in HttpClient and replace with thread-safe implementations where necessary.
> Oleg

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


[jira] Resolved: (HTTPCLIENT-824) Review the use of BaiscHttpParams and HttpProtocolProcessor in HttpClient

by JIRA jira@apache.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message


     [ https://issues.apache.org/jira/browse/HTTPCLIENT-824?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Oleg Kalnichevski resolved HTTPCLIENT-824.
------------------------------------------

    Resolution: Fixed

Fixed in SVN trunk

Oleg

> Review the use of BaiscHttpParams and HttpProtocolProcessor in HttpClient
> -------------------------------------------------------------------------
>
>                 Key: HTTPCLIENT-824
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-824
>             Project: HttpComponents HttpClient
>          Issue Type: Task
>          Components: HttpClient
>            Reporter: Oleg Kalnichevski
>             Fix For: 4.1.0
>
>
> Review the use of BaiscHttpParams and HttpProtocolProcessor in HttpClient and replace with thread-safe implementations where necessary.
> Oleg

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...