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

request entity too large on json import #163

Closed
Stefal opened this issue Aug 26, 2023 · 7 comments
Closed

request entity too large on json import #163

Stefal opened this issue Aug 26, 2023 · 7 comments

Comments

@Stefal
Copy link

Stefal commented Aug 26, 2023

Describe the bug

I can't import a kresus json backup

Context

  • Hardware: Old computer with proxmox
  • YunoHost version: 11.2.3
  • I have access to my server: *Through SSH | through the webadmin |

Steps to reproduce

  • If the error occurs in your browser, explain what you did:
    1. Go to 'Kresus Import'
    2. Import a json file with a file size > 1.69MB'
    3. Click on import
    4. See error
      image

In the web browser console I see these messages :

An error has occurred with the following information:
        - message: ?
        - stack: no stack
        - stringified: {"message":"?","shortMessage":"?"}

And

État  413  Request Entity Too Large

Expected behavior

Working import

@Stefal
Copy link
Author

Stefal commented Aug 26, 2023

Just for a quick test, I've removed some transaction from my backup to get a 1.2MB json file, and now I can import it.

What setting should I edit to be able to import my full backup ?

@nicofrand
Copy link
Collaborator

Hi!

client_max_body_size 8M;
is supposed to be enough.

I'll double check if something is missing though, next week.

@nicofrand
Copy link
Collaborator

Do you have more logs?

@Stefal
Copy link
Author

Stefal commented Aug 27, 2023

After some trial and errors, I've added client_max_body_size 20M; in the html section of /etc/nginx/nginx.conf and the import was successful.

@Stefal
Copy link
Author

Stefal commented Aug 27, 2023

I'm not sure it's a good idea to close this issue. Something should be done so nobody would have this problem in the future.

@nicofrand
Copy link
Collaborator

nicofrand commented Aug 27, 2023 via email

@bnjbvr
Copy link
Contributor

bnjbvr commented Aug 29, 2023

In any case, if you see any possible improvements @Stefal, feel free to make a patch! We welcome contributions :-)

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

No branches or pull requests

3 participants