Re: [I] Remove export functionality from UI / Public API [airflow]
Shally-Katariya commented on issue #59840: URL: https://github.com/apache/airflow/issues/59840#issuecomment-3721709966 Thanks for checking this. When I initially investigated, I didn’t find any active export functionality. I proceeded with limiting public API list responses for connections and variables to support the broader security goal, but there’s no remaining export-related work needed here. Agree that this issue can be closed. -- 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]
Re: [I] Remove export functionality from UI / Public API [airflow]
nhuantho commented on issue #59840: URL: https://github.com/apache/airflow/issues/59840#issuecomment-3721649613 @potiuk When I checked this issue, the connection has never had export functionality. Then I checked the variable, and I saw [57594] (https://github.com/apache/airflow/pull/57594), which resolved this issue. So I think that we can close this issue. -- 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]
Re: [I] Remove export functionality from UI / Public API [airflow]
potiuk commented on issue #59840: URL: https://github.com/apache/airflow/issues/59840#issuecomment-3694304914 Sure - no need to ask for permission or provide plan - I know many AIs do that, but it adds additional effort to review plans like that - adds extra noise, and holds you from starting doing it until somoene responds - we do not **want** to block you from trying - and even if it is bad attempt, it's better to comment and review things when PR is opened, so you can proceed straight to producing PR without asking anyone for comment or accepting your plan. We prefer people who are able to act on their own (even if they use AI to help them) - also as long as they are not copy&pasting stuff generated by AI verbatim but understand the change and learn from the feedback they receieve (which is best when there is a PR). -- 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]
Re: [I] Remove export functionality from UI / Public API [airflow]
Shally-Katariya commented on issue #59840: URL: https://github.com/apache/airflow/issues/59840#issuecomment-3694145184 Hi @potiuk I’d like to take this issue up. Plan: - Remove export actions for Connections and Variables from the UI - Replace export options with an explanatory message indicating exports are only allowed via the local CLI - Ensure import functionality remains unchanged - Verify corresponding Public API export endpoints are disabled/removed where applicable I’ll start by reproducing the current behavior locally and will share progress here. Please let me know if this approach aligns with the intended direction. -- 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]
[I] Remove export functionality from UI / Public API [airflow]
potiuk opened a new issue, #59840: URL: https://github.com/apache/airflow/issues/59840 ### Body Make sure all export functionality for connections and variables is removed from the UI and replaced with explanation that export can only be available via local CLI. Make sure import is available. ### Committer - [x] I acknowledge that I am a maintainer/committer of the Apache Airflow project. -- 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]
