[ 
https://issues.apache.org/jira/browse/TIKA-3326?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17302813#comment-17302813
 ] 

ASF GitHub Bot commented on TIKA-3326:
--------------------------------------

tballison merged pull request #416:
URL: https://github.com/apache/tika/pull/416


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Code cleaning and Javadoc
> -------------------------
>
>                 Key: TIKA-3326
>                 URL: https://issues.apache.org/jira/browse/TIKA-3326
>             Project: Tika
>          Issue Type: Improvement
>          Components: server
>    Affects Versions: 2.0.0, 1.25
>            Reporter: Subhajit Das
>            Priority: Trivial
>
> TikaResource Uses  String key : httpHeaders.keySet() to iterate over header 
> keys, and then inside processHeaderConfig(...), the first header value is 
> fetched, and stored in "val" variable, for further usage.
>  
> As, this method (processHeaderConfig) should be logically called (and 
> practically) per header value pair, it would be cleaner to extract the header 
> value to outside.
>  
> Additionally, few missing javadocs added. Updated javadocs for changed 
> methods as well.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to