[ 
https://issues.apache.org/jira/browse/HTTPCLIENT-1037?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12973886#action_12973886
 ] 

Oleg Kalnichevski commented on HTTPCLIENT-1037:
-----------------------------------------------

I took a closer look at the classes that implement ContentDescriptor and they 
all return non-null values for mimeType and transferEncoding. I was wrong about 
javadocs being wrong. 

FormBodyPart is simply coded defensively. I do not think we have any problem at 
all. Feel free to change FormBodyPart or re-phrase javadocs of 
ContentDescriptor, if you deem it necessary.

Oleg

> FormBodyPart code does not agree with ContentDescriptor Javadoc wrt 
> nullability of mimeType and transferEncoding
> ----------------------------------------------------------------------------------------------------------------
>
>                 Key: HTTPCLIENT-1037
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-1037
>             Project: HttpComponents HttpClient
>          Issue Type: Bug
>          Components: HttpMime
>            Reporter: Sebb
>
> The FormBodyPart does not agree with ContentDescriptor Javadoc wrt 
> nullability of mimeType and transferEncoding:
> The code in FormBodyPart explicitly allows mimeType and transferEncoding to 
> be null, in which case the relevant header is not generated.
> This is useful behaviour, as the headers are not necessaruly needed.
> However the bahaviour disagrees with the Javadoc in the ContentDescriptor 
> interface - null is not allowed.
> Also, AbstractContentBody does not allow mime-type to be null.

-- 
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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to