[
https://issues.apache.org/jira/browse/AIRFLOW-4295?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16847938#comment-16847938
]
ASF GitHub Bot commented on AIRFLOW-4295:
-----------------------------------------
kaxil commented on pull request #5313: AIRFLOW-4295. Make comparison of
attribute `method` case insensitive …
URL: https://github.com/apache/airflow/pull/5313
----------------------------------------------------------------
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:
[email protected]
> 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
> Fix For: 1.10.4
>
> 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)
