Fix [Artifacts] disable download for not allowed prefixes in artifact…

…/dataset/model path (#2486)
mlrun · May 30, 2024 · 2742ba1 · 2742ba1
1 parent d9ac1cc
commit 2742ba1
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 0 deletions.
diff --git a/src/components/Datasets/datasets.util.js b/src/components/Datasets/datasets.util.js
@@ -256,6 +256,7 @@ export const generateActionsMenu = (
       {
         hidden: menuPosition === ACTION_MENU_PARENT_ROW_EXPANDED,
         label: 'Download',
+        disabled: !isTargetPathValid,
         icon: <DownloadIcon />,
         onClick: dataset => {
           dispatch(

diff --git a/src/components/Files/files.util.js b/src/components/Files/files.util.js
@@ -238,6 +238,7 @@ export const generateActionsMenu = (
       {
         label: 'Download',
         hidden: menuPosition === ACTION_MENU_PARENT_ROW_EXPANDED,
+        disabled: !isTargetPathValid,
         icon: <DownloadIcon />,
         onClick: file => {
           dispatch(

diff --git a/src/components/ModelsPage/Models/models.util.js b/src/components/ModelsPage/Models/models.util.js
@@ -325,6 +325,7 @@ export const generateActionsMenu = (
       {
         label: 'Download',
         hidden: menuPosition === ACTION_MENU_PARENT_ROW_EXPANDED,
+        disabled: !isTargetPathValid,
         icon: <DownloadIcon />,
         onClick: model => {
           dispatch(