[
https://issues.apache.org/jira/browse/AIRFLOW-4295?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16843476#comment-16843476
]
ASF subversion and git services commented on AIRFLOW-4295:
----------------------------------------------------------
Commit 58105308d8a5d13f6593e2a9fcfd08cf5dc067e8 in airflow's branch
refs/heads/master from Karl Andersson
[ https://gitbox.apache.org/repos/asf?p=airflow.git;h=5810530 ]
[AIRFLOW-4295] Make `method` case insensitive in HTTPHook (#5173)
Make the method: `run` in the HttpHook compare the attribute: 'method' in a
case insensitive way.
This resolves the issue where a Httphook created with parameter `method='get'`
would not be
treated as a GET-request in the run method and the attribute `params`would be
omitted in the Http request.
> HttpHook allows to create methods in lower case, e.g 'get' but correct usage
> requires method to be in upper case
> ----------------------------------------------------------------------------------------------------------------
>
> Key: AIRFLOW-4295
> URL: https://issues.apache.org/jira/browse/AIRFLOW-4295
> Project: Apache Airflow
> Issue Type: Bug
> Components: hooks
> Affects Versions: 1.10.2
> Reporter: Karl Andersson
> Assignee: Karl Andersson
> Priority: Minor
> Original Estimate: 1h
> Remaining Estimate: 1h
>
> The HttpHook except any value for variable `method` in the constructor.
> However, creating a hook with method = 'get' will cause the run-function not
> to treat it as a GET reqest and the `params` parameter will not be used. This
> is confusing as the request will just omit the `params` parameter.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
