potiuk commented on pull request #9333: URL: https://github.com/apache/airflow/pull/9333#issuecomment-645378386
I think all comments handled in the last two fixups. We have now mount_point taken from the path, explicit support for vaults/vault URL schemas (on top of http/https). _VaultClient is now moved to separate package and excluded from doc. Token is taken from password. I still left "overrideable" extras in the constructor (for non-sensitive fields) as I think it's really useful for anyone trying to use the hook. ---------------------------------------------------------------- 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: us...@infra.apache.org
