Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: show the network error alert when upload large file failed. #1782

Closed
1 task done
Chunshan-Theta opened this issue Aug 2, 2024 · 5 comments
Closed
1 task done
Labels
bug Something isn't working

Comments

@Chunshan-Theta
Copy link
Contributor

Chunshan-Theta commented Aug 2, 2024

Is there an existing issue for the same bug?

  • I have checked the existing issues.

Branch name

main

Commit ID

240802twncutheta

Other environment information

No response

Actual behavior

I got the network error alert when trying to upload the files.
large file -0

I checked the dev tool, and the true reason is the file's size is too large
large file -1

  1. Maybe the alert needs to update the text?
  2. Do We have another way could upload large files?

Expected behavior

No response

Steps to reproduce

upload a class textbook.

Additional information

No response

@Chunshan-Theta Chunshan-Theta added the bug Something isn't working label Aug 2, 2024
@KevinHuSh
Copy link
Collaborator

Check the nginx configuration.
BTW, demo site didn't support file larger than 10M.

@Chunshan-Theta
Copy link
Contributor Author

Chunshan-Theta commented Aug 2, 2024

the alert text had already changed! thank you for your quick action.
large file -2

I followed your tips on updating the configuration of nginx. and restart the docker compose (down & up -d)
nginx.conf:
large file -6
In container:
large file -4

but still show the 413 error

{data: null, retcode: 100, retmsg: "<RequestEntityTooLarge '413: Request Entity Too Large'>"}

my file info:
large file -5

@KevinHuSh
Copy link
Collaborator

Set MAX_CONTENT_LENGTH=1000000000 in docker/.env

@Chunshan-Theta
Copy link
Contributor Author

large file -7
added, But I am sorry it did not work.
Error 413 was returned.

@Chunshan-Theta
Copy link
Contributor Author

I added it in docker-compose.yml, It work :)

large file -8

cike8899 added a commit to cike8899/ragflow that referenced this issue Sep 29, 2024
KevinHuSh pushed a commit that referenced this issue Sep 29, 2024
…ploading a file that was too large #1782 (#2654)

### What problem does this PR solve?

_Briefly describe what this PR aims to solve. Include background context
that will help reviewers understand the purpose of the PR._

### Type of change

- [x] Bug Fix (non-breaking change which fixes an issue)
- [ ] New Feature (non-breaking change which adds functionality)
- [ ] Documentation Update
- [ ] Refactoring
- [ ] Performance Improvement
- [ ] Other (please describe):
Halfknow pushed a commit to Halfknow/ragflow that referenced this issue Nov 11, 2024
…ploading a file that was too large infiniflow#1782 (infiniflow#2654)

### What problem does this PR solve?

_Briefly describe what this PR aims to solve. Include background context
that will help reviewers understand the purpose of the PR._

### Type of change

- [x] Bug Fix (non-breaking change which fixes an issue)
- [ ] New Feature (non-breaking change which adds functionality)
- [ ] Documentation Update
- [ ] Refactoring
- [ ] Performance Improvement
- [ ] Other (please describe):
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants