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

Solving a caching issue of minio + small improvements when creating users #4

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

fversaci
Copy link

  • There was an issue when uploading large files with POST to the minio server: without the "filename" field data is considered small and cached in memory, with a hard limit of 16 MB. I've added the filename" field to force minio to cache data to disk during the upload, thus bypassing the 16 BM limit.
  • I've also adopted the standard minio server in the docker-compose, which now works fine.
  • Small changes/fix when creating users

the minio server to cache data on disk, bypassing the 16MB limit
of in-memory caching.
- Standard, non-customized, minio server is adopted in docker-compose
ajspeck pushed a commit to ajspeck/zotero-prime that referenced this pull request Apr 23, 2020
Via the 'locale' query parameter

Closes FiligranHQ#4
@shodr
Copy link

shodr commented Feb 15, 2021

Hi, fversaci
I was trying to implement regular minio, with your changes, but I get an error "There were headers present in the request which were not signed", I get this error whichever version of minio docker image I use (latest or the ones form the date you posted your solution. Tried modifying function "getDownloadDetails", or to point directly at minio server (without redirect to 8082), but cannot get around the error. I can generate download links through mino browser or AWS cli in app-zotero, but not through zotero client.
Do you, anybody else know a solution. (I am behind corporate proxy if its relevant). I believe the issue ist with the header X-Amz-Content-Sha256: which is signed as an empty string (hash: e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855), but cannot get the SDK to ignore it when generating link).
tnx.
tnx

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.

2 participants