search

irods-contrib / metalnx-web

Metalnx Web Application
https://metalnx.github.io/
BSD 3-Clause "New" or "Revised" License
36 stars 36 forks source link

Unable to download more than 100MB file #163

Closed kript closed 3 years ago

kript commented 3 years ago

Despite #143 being resolved, I find that even with metalnx.download.limit=5120 set in metalnx.properties the UI still blocks downloads of larger than 100MB.

michael-conway commented 3 years ago

I'm seeing it here...will fold into issues I'm looking at today

michael-conway commented 3 years ago

can we check the version? I was able to test a fail with 100 set and then success when I pushed to 6000 in that property...

image

michael-conway commented 3 years ago

...in the about tab by the user dropdown on the upper right

michael-conway commented 3 years ago

ps for my own testing I had this in metalnx.properties...

metalnx.download.limit=6000

michael-conway commented 3 years ago

Just did upload/download of 3GB no prob...

image

kript commented 3 years ago

can we check the version? I was able to test a fail with 100 set and then success when I pushed to 6000 in that property...

image

huh, apparently we have 2.1.0-RELEASE however we built the docker container from 20311a55e66a091f078cd85f1ddc2ad9bdb4ca53 which might obfuscate the above tag? we'll look into getting the latest.

michael-conway commented 3 years ago

Hey John I think that there are a few issues together fixed that might warrant pushing a test container outright. I can throw one up on dockerhub if that helps.

On Wed, Oct 7, 2020 at 4:49 AM John Constable notifications@github.com wrote:

can we check the version? I was able to test a fail with 100 set and then success when I pushed to 6000 in that property...

[image: image] https://user-images.githubusercontent.com/1097261/95230454-eb2aaf80-07cf-11eb-9c04-9caf5c1a92bc.png

huh, apparently we have 2.1.0-RELEASE however we built the docker container from 20311a55e66a091f078cd85f1ddc2ad9bdb4ca53 which might obfuscate the above tag? we'll look into getting the latest.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/irods-contrib/metalnx-web/issues/163#issuecomment-704790630, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIL4LJRMSEK3EYMKGVCY33SJQTPXANCNFSM4SFYRUWQ .

kript commented 3 years ago

Thanks Mike, worth a try, or we can build from a commit you recommend?

michael-conway commented 3 years ago

I can just throw something up on my dockerhub if you'd like...

kript commented 3 years ago

Is building from master recommended, or is there a particular commit?

On 8 October 2020 at 13:59:34, Mike Conway (notifications@github.com) wrote:

I can just throw something up on my dockerhub if you'd like...

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

michael-conway commented 3 years ago

I pushed michaelconway/metalnx:latest to reflect current master