search

aws-samples / aws-genai-llm-chatbot

A modular and comprehensive solution to deploy a Multi-LLM and Multi-RAG powered chatbot (Amazon Bedrock, Anthropic, HuggingFace, OpenAI, Meta, AI21, Cohere, Mistral) using AWS CDK on AWS
https://aws-samples.github.io/aws-genai-llm-chatbot/
MIT No Attribution
967 stars 284 forks source link

PDF upload fails with spaces in filename #452

Open ystoneman opened 2 months ago

ystoneman commented 2 months ago

I tried uploading a PDF with spaces in the name ("Executive Order on the Safe, Secure, and Trustworthy Development and Use of Artificial Intelligence _ The White House.pdf") using the Chatbot's UI. This failed, and the UI showed me a generic error ("Uploading files Upload failed"). Chrome's dev tools showed me it was an S3/CloudFront issue:

"Failed to load resource: the server responded with a status of 403 (Forbidden)."

Screenshot 2024-04-14 at 4 46 00 PM

Replacing the spaces with underscores in the name ("Executive_order_AI.pdf") made the error go away and the file upload.

To improve UX I would either:

  1. Improve the UI's error message to tell users that the filename contains invalid characters (like spaces) and tells them the filename requirements.
  2. Or even better, automatically normalize the filename before upload by replacing spaces with underscores, truncating if needed, and applying other necessary transformations for S3 compatibility.
azaylamba commented 2 months ago

Actually, the issue is not with using spaces, the issue is with having comma(,) in the file name.

azaylamba commented 2 months ago

This can be closed now as the fix has been merged to main.

github-actions[bot] commented 2 weeks ago

This issue is stale because it has been open for 60 days with no activity.