ciancolo commented on pull request #17125: URL: https://github.com/apache/airflow/pull/17125#issuecomment-884023597
Oh really?! I didn't know. Thank you for the hint. I was thinking, what if we removed completely the conversion? I mean the admissible value for that `Verify` field with json standard are: - true - false - string to the cert path If the user will use for example `"False"` (as string) as value to `Verify`, in my opinion, the `Tableau Hook` should use it as a string and search the certificate in the path (of course it will fail). For me, it is not necessary to convert the parameter to bool in case the user uses it in the wrong way. What do you think? -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
