update upload-from-url role resource

halo-dev · Aug 21, 2024 · eddbf10 · eddbf10
1 parent 9526cb4
commit eddbf10
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 9 deletions.
diff --git a/...tion/src/main/java/run/halo/app/core/extension/service/impl/DefaultAttachmentService.java b/...tion/src/main/java/run/halo/app/core/extension/service/impl/DefaultAttachmentService.java
@@ -143,7 +143,6 @@ public Mono<URI> getSharedURL(Attachment attachment, Duration ttl) {
     @Override
     public Mono<Attachment> uploadFromUrl(@NonNull URL url, @NonNull String policyName,
         String groupName, String filename) {
-        // TODO validate url
         var uri = URI.create(url.toString());
         AtomicReference<MediaType> mediaTypeRef = new AtomicReference<>();
         AtomicReference<String> fileNameRef = new AtomicReference<>(filename);
@@ -176,9 +175,7 @@ private static String getExternalUrlFilename(URI externalUrl, HttpHeaders httpHe
             var path = externalUrl.getPath();
             fileName = Paths.get(path).getFileName().toString();
         }
-        if (!FileNameUtils.hasFileExtension(fileName)) {
-            // TODO get file extension from media type
-        }
+        // TODO get file extension from media type
         return fileName;
     }
 

diff --git a/application/src/main/resources/extensions/role-template-attachment.yaml b/application/src/main/resources/extensions/role-template-attachment.yaml
@@ -17,12 +17,14 @@ rules:
   - apiGroups: [ "api.console.halo.run" ]
     resources: [ "attachments" ]
     verbs: [ "*" ]
+  - apiGroups: [ "api.console.halo.run" ]
+    resources: [ "attachments/upload-from-url" ]
+    verbs: [ "create" ]
   - apiGroups: [ "" ]
     resources: [ "settings" ]
     verbs: [ "get" ]
   - nonResourceURLs: [
-      "/apis/api.console.halo.run/v1alpha1/attachments/upload",
-      "/apis/api.console.halo.run/v1alpha1/attachments/-/upload-from-url"
+      "/apis/api.console.halo.run/v1alpha1/attachments/upload"
     ]
     verbs: [ "create" ]
 ---

diff --git a/application/src/main/resources/extensions/role-template-uc-content.yaml b/application/src/main/resources/extensions/role-template-uc-content.yaml
@@ -114,6 +114,6 @@ rules:
   - apiGroups: [ "uc.api.content.halo.run" ]
     resources: [ "attachments" ]
     verbs: [ "create", "update", "delete" ]
-  - nonResourceURLs: [
-    "/apis/api.console.halo.run/v1alpha1/attachments/-/upload-from-url"
-  ]
+  - apiGroups: [ "uc.api.content.halo.run" ]
+    resources: [ "attachments/upload-from-url" ]
+    verbs: [ "create" ]