Merge pull request #155 from microsoft/PSL-BUG-12797

fix: 0 byte/kb size file not allowed.

Author: Roopan-Microsoft

App/backend-api/Microsoft.GS.DPS.Host/API/KernelMemory/KernelMemory.cs

@@ -49,6 +49,17 @@ DPS.API.KernelMemory kernelMemory
                                                                                  Summary = $"{fileExtension} file is Unsupported file type" });
                     }
 
+                    // Checking File Size: O byte/kb file not allowed
+                    if (file == null || file.Length == 0)
+                    {
+                        return Results.BadRequest(new DocumentImportedResult()
+                        {
+                            DocumentId = string.Empty,
+                            MimeType = contentType,
+                            Summary = "The file is empty and cannot be uploaded. Please select a valid file."
+                        });
+                    }
+
                     var result = await kernelMemory.ImportDocument(fileStream, file.FileName, contentType);
 
                     //Return HTTP 202 with Location Header

App/frontend-app/src/components/uploadButton/uploadButton.tsx

@@ -24,7 +24,7 @@ import { getFileTypeIconProps } from "@fluentui/react-file-type-icons";
 const UploadDocumentsDialog = () => {
   const [isOpen, setIsOpen] = useState(false);
   const [uploadingFiles, setUploadingFiles] = useState<
-    { name: string; progress: number; status: string }[]
+    { name: string; progress: number; status: string; errorMsg:string }[]
   >([]);
   const [isUploading, setIsUploading] = useState(false);
 
@@ -35,6 +35,7 @@ const UploadDocumentsDialog = () => {
       name: file.name,
       progress: 0,
       status: "uploading",
+      errorMsg : ""
     }));
     setUploadingFiles((prev) => [...prev, ...newFiles]);
 
@@ -52,18 +53,19 @@ const UploadDocumentsDialog = () => {
         setUploadingFiles((prev) =>
           prev.map((f, index) =>
             index === prev.length - acceptedFiles.length + i
-              ? { ...f, progress: 100, status: "success" }
+              ? { ...f, progress: 100, status: "success", errorMsg: "" }
               : f
           )
         );
-      } catch (error) {
-        console.error("Error uploading file:", error);
+      } catch (error:any) {
+        const errorMessage = error.message.replace(/^Error:\s*/, ""); // Remove "Error: " prefix
+        const parsedError = JSON.parse(errorMessage);
 
         // Update file status to error
         setUploadingFiles((prev) =>
           prev.map((f, index) =>
             index === prev.length - acceptedFiles.length + i
-              ? { ...f, progress: 100, status: "error" }
+              ? { ...f, progress: 100, status: "error", errorMsg: parsedError.summary}
               : f
           )
         );
@@ -188,13 +190,17 @@ const UploadDocumentsDialog = () => {
                   )}
                 </div>
                 <ProgressBar value={file.progress} />
-                <span>
+                <span
+                style={{
+                  color: file.status === "error" ? "red" : "inherit", // Apply red color only for error messages
+                }}>
                   {file.status === "uploading"
                     ? "Uploading..."
                     : file.status === "success"
                     ? "Upload complete"
-                    : "Upload failed"}
+                    : file.errorMsg}
                 </span>
+                <span>{}</span>
               </div>
             ))}
           </DialogContent>