vincbeck commented on PR #55687:
URL: https://github.com/apache/airflow/pull/55687#issuecomment-3318908619
> Feels like filter_authorized_{foo} methods could be consolidated into a
generic "filter list" method, but I don't have the whole plan/picture here so
maybe that's not worth the effort.
>
> Code LGTM
You're right but we might want to keep it because when it might make our
life easier when we will "teamify" AWS auth manager. If not, we could indeed
factorize the code a bit. Let's wait and see :)
--
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]
