[
https://issues.apache.org/jira/browse/AIRFLOW-4295?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16845986#comment-16845986
]
ASF GitHub Bot commented on AIRFLOW-4295:
-----------------------------------------
karan5291 commented on pull request #5313: AIRFLOW-4295. Make comparison of
attribute `method` case insensitive …
URL: https://github.com/apache/airflow/pull/5313
This PR addresses the following [Airflow
Jira](https://issues.apache.org/jira/browse/AIRFLOW-4295)
(It is a reopened version of https://github.com/apache/airflow/pull/5173)
### Description
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.
### Tests
- Unit tests: `test_hook_with_method_in_lowercase`
----------------------------------------------------------------
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)
