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

Not possible to download proof of delivery #3610

Closed
rayon9coop opened this issue May 3, 2023 · 4 comments
Closed

Not possible to download proof of delivery #3610

rayon9coop opened this issue May 3, 2023 · 4 comments
Labels

Comments

@rayon9coop
Copy link

bug

@rayon9coop rayon9coop added the bug label May 3, 2023
@alexsegura
Copy link
Member

In this case, it's because there are many images, and the browserless server is responding "Payload Too Large"

It seems to have been fixed for a long time (browserless/browserless#150), so it should be possible to fix it by incrementing the MAX_PAYLOAD_SIZE env var (defaults to 5mb)

@Paul-Eraman-CoopCycle
Copy link
Contributor

Hey, so it looks like photos are accesable in two places:

  1. here, where a pdf is printed using a 3rd party service, which here has had an error due to the total size of the photos
  2. A shop dashboard where there is a history of the services of the shop, where there is a camera icon that will download a zip file with all the photos - this should still be working.

So while it looks like we can solve this issue today (to be confirmed) the best practices here is to have these tasks associated with a shop, which also avoids this issue

@alexsegura
Copy link
Member

Struggling the restart browserless with new env var in production because of docker/docker-py#3113

@alexsegura
Copy link
Member

Fixed by increasing MAX_PAYLOAD_SIZE to 20mb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants