[HTTPCORE-223] Thread-safety improvements

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 5.0-alpha1
• Component/s: HttpCore
• Assignee: Unassigned
• Reporter: Sebb
• Created: 22/Mar/10 23:00
• Updated: 14/Feb/15 13:12
• Resolved: 14/Feb/15 13:12

Description

Thread-safety / isolation improvements to be considered for version 5.0
========================================================

AbstractHttpEntity
has 3 protected fields, however there are public getters/setters for
all of the fields, so it seems to me that the fields should be
private. That would allow synch. to be added later if necessary.

HttpEntityWrapper

• wrappedEntity should be final.

AbstractHttpMessage

• headerGroup should be final
• params should be private

BasicHttpProcessor
requestInterceptors and responseInterceptors should be final
Should also be private, otherwise subclasses can subvert the non-null condition