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

The file is incomplete (105%). Try uploading again. #899

Open
bits4life opened this issue Oct 30, 2022 · 22 comments
Open

The file is incomplete (105%). Try uploading again. #899

bits4life opened this issue Oct 30, 2022 · 22 comments

Comments

@bits4life
Copy link

Hi community,

I am trying to setup sharry on my k3s cluster and it seems the work out fine except that for every upload there is an error about incomplete file upload (ranging from 103 - 105 %)
image

Some details on my setup :

  • deployment is using the dockerhub eikek0/sharry latest image tag
  • kubernetes node is arm64 based
  • config: |
    sharry.restserver {
    base-url = "https://sharry.xxxmydomainxxx.be"
    bind {
    address = "0.0.0.0"
    port = {{ .Values.service.main.ports.http.port }}
    }
    backend {
    auth {
    fixed {
    enabled = true # set to true to enable this auth provider
    user = "admin"
    password = "admin"
    order = 10
    }
    }
    }
    }

I set log-level to debug and could see the following errors :
2022.10.30 11:46:40:0000 [io-comp...] [WARN ] org.http4s.blazecore.util.IdentityWriter - Will not write more bytes than what was indicated by the Content-Length header (347753)
2022.10.30 11:46:40:0000 [io-comp...] [ERROR] org.http4s.blaze.server.Http1ServerStage$$anon$1 - Error writing body

Not sure if this is related.

I tried different versions, also changed to file store VS database, tried disabling the checksum but nothing seems to make a difference.
Any idea how to further troubleshoot would be appreciated.

Best regards,
Nico

@eikek
Copy link
Owner

eikek commented Oct 30, 2022

Hey @bits4life , I'm not sure really where to look. It seems that uploading did not work correctly, but can't really tell. I tried it here, but could not reproduce it (but not kubernetes and not arm64). The warning means that there exists more bytes than the reported content-length of the file. This could be a bad content-length header while uploading (not really likely to me) or some misbehaviour when chunking or some misbehaviour when storing chunks.

Does it happen with any file or is there some kind of pattern? Does it change without a reverse proxy?

@bits4life
Copy link
Author

bits4life commented Oct 31, 2022 via email

@eikek
Copy link
Owner

eikek commented Nov 3, 2022

That sounds a bit like that the text files are rather small and can be uploaded in one chunk completely whereas other files are uploaded in mutlliple chunks, could this be? You could try setting log level to Debug or even Trace for logger sharry.restserver.routes. This should print out a bit more when uploading (or processing requests in general).

Does it also occur when using a "normal" upload with e.g. curl like this example? The webapp does it a bit differently, if the curl command would work, at least then it is likely in the chunked upload somewhere. I cannot reproduce it though :/

@insuusvenerati
Copy link

I'm also running into this issue while on kubernetes. The logs i'm getting on a fresh instance: https://hastebin.com/veribikapu.yaml

@insuusvenerati
Copy link

I confirmed that the issue doesn't happen when uploading using curl.

@insuusvenerati
Copy link

Here's an example file that has the issue:

{
    "id": "15vrEnEHMjd-n9TY1jBCnfF-aefv9ZWzym3-skSivJBWmc",
    "name": null,
    "aliasId": null,
    "aliasName": null,
    "validity": 604800000,
    "maxViews": 30,
    "password": false,
    "descriptionRaw": "",
    "description": "",
    "created": 1671740820902,
    "publishInfo": null,
    "files": [
        {
            "id": "9FmbL6W29ek-9XFR6Wv25uu-RGATEJKSMjA-vSwjC5qU7Sr",
            "filename": "305752627_5281399795241558_427954128394578030_n.jpg",
            "size": 45104,
            "mimetype": "image/jpeg",
            "checksum": "",
            "storedSize": 47154
        }
    ]
}

@cerealconyogurt
Copy link

Hello @insuusvenerati, I am having the same problem. Did you find the problem?

@VitusE
Copy link

VitusE commented Mar 4, 2023

Same here.
Any anyone accomplished something yet?

@eikek
Copy link
Owner

eikek commented Mar 4, 2023

Hi there, totally lost this issue…. But sadly I cannot reproduce 😟. Are there any other details that could help to reproduce? Did some/most of you changed config values, like file storage? From the original description it seems not affected by the file backend, but any info might help :).

@insuusvenerati
Copy link

Hi there, totally lost this issue…. But sadly I cannot reproduce 😟. Are there any other details that could help to reproduce? Did some/most of you changed config values, like file storage? From the original description it seems not affected by the file backend, but any info might help :).

You're deploying in k8s?

@eikek
Copy link
Owner

eikek commented Mar 5, 2023

You're deploying in k8s?

No, don't know k8s very well and I need to reproduce it without to better analyze what's going on.

@cerealconyogurt
Copy link

Hello, I just returned from a long trip. I still have the problem:

image

For some reason the txt file does not show the problem. Not sure why.
What should I do to support the problem solving process?

I will start a fresh install as a first step...

@eikek
Copy link
Owner

eikek commented Mar 25, 2023

That's a good question. Maybe a try is to record all the requests when uploading one such file that fails and send me the HAR export from the browser.

@cerealconyogurt
Copy link

That's a good question. Maybe a try is to record all the requests when uploading one such file that fails and send me the HAR export from the browser.

@eikek, find attached the HAR file

har.zip

I had to zip it since the .har file was not recognized by GitHub as valid archive

@eikek
Copy link
Owner

eikek commented Mar 31, 2023

Hi @cerealconyogurt thank you a lot for this! I'll try to have a look over the weekend.

@PietHelzel
Copy link

PietHelzel commented Apr 26, 2023

@eikek Having the same issue. Because I saw it happen to many files (like pdf), I just cut out stuff from the pdf file to see what the condition for it happening is. I ended up reducing it to a single character file (see file below). So it seems some weird bytes cause this error. Interestingly, if you add for example an "a" after that character, it gets cut out in the file after uploading to sharry. In a hex editor this specific character shows up as the bytes EC, 8F.
No idea what to make of all this, maybe it helps.
test.txt

@PietHelzel
Copy link

Just tested some older versions, it seems to work in 1.10.0, but not in 1.11.0 and newer.

@eikek
Copy link
Owner

eikek commented Apr 28, 2023

Thanks @PietHelzel for all the info. I'm sorry that it's currently a bit hard for me to find time. Also, the fact that I cannot reproduce it locally makes it quite hard to figure out. But it's a great thing to know that it has been working in an earlier version.

@cerealconyogurt
Copy link

Just tested some older versions, it seems to work in 1.10.0, but not in 1.11.0 and newer.

I can confirm the version 1.10.0 works flawless

@PietHelzel
Copy link

PietHelzel commented May 2, 2023

@eikek Good news, it seems like filesystem storage on the newest version works too, it only affects the database storage

@phires
Copy link

phires commented Jul 4, 2023

Hi, chiming in on this issue.
Did a complete new and clean installation on a fresh Ubuntu 22.04 server using the current .deb file.
Reverse Proxy is set up with nginx using the example configuration.
Upload of a .mp4 file is reported with "The file is incomplete (106%). Try uploading again."
Using the file-system storage works flawlessly, it's only with database storage. Haven't tested minio.

@Ravenbs
Copy link

Ravenbs commented Apr 29, 2024

Same issue, reported details in redundant thread: #1327

I can confirm the tipp the others said. Its working with file storage and not with H2 storage.

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

8 participants