这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@timothycarambat
Copy link
Member

connect #169
resolves #427

Screen Shot 2023-12-12 at 4 00 54 PM Screen Shot 2023-12-12 at 4 01 12 PM Screen Shot 2023-12-12 at 4 01 17 PM

@review-agent-prime
Copy link

server/utils/files/multer.js

There is a repeated pattern in the code where the destination for file uploads is being set. This logic can be extracted into a separate function to improve the readability and maintainability of the code.
Create Issue
See the diff
Checkout the fix

    function getUploadDestination(dir) {
      return process.env.NODE_ENV === "development"
        ? path.resolve(__dirname, dir)
        : path.resolve(process.env.STORAGE_DIR, dir);
    }
git fetch origin && git checkout -b ReviewBot/Impro-x6z7nyv origin/ReviewBot/Impro-x6z7nyv

@timothycarambat timothycarambat merged commit d4f4d85 into master Dec 13, 2023
@timothycarambat timothycarambat deleted the non-latin-filenames branch December 13, 2023 00:07
cabwds pushed a commit to cabwds/anything-llm that referenced this pull request Jul 3, 2025
… and chat (Mintplex-Labs#432)

patch: fix non-latin filenames being messed up during upload and chat
connect Mintplex-Labs#169
resolves Mintplex-Labs#427
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Filenames that are not in latin script are displayed as garbage

2 participants