Taragolis commented on PR #33602: URL: https://github.com/apache/airflow/pull/33602#issuecomment-1697191415
> I wil do it after we merge this PR I would rather say `if we merge this PR`, I have personal concern of: 1. How it work exclusively for integration with Docker. Implementation for ECR not exclusive, it might be implemented for other Container/Docker Registry providers who could provide temporal access to registry 2. Naming, still not sure that this part should named as `protocols`, for me it so broad. If this solution would be accepted, we might finally remove exclusive implementation AWS IAM authentication in PostgresHook and MySQLHook by the same method. -- 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]
