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

Subhajit Das updated TIKA-3326:
-------------------------------
    Description: 
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.

> 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