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

Strange logging for TUS Upload in oCIS #9121

Closed
dragotin opened this issue Oct 5, 2021 · 0 comments
Closed

Strange logging for TUS Upload in oCIS #9121

dragotin opened this issue Oct 5, 2021 · 0 comments
Milestone

Comments

@dragotin
Copy link
Contributor

dragotin commented Oct 5, 2021

I was trying to sync up a > 500 MB file to oCIS via Linux desktop client, version 2.9.0

The server pretends that this fails due to owncloud/ocis#1205 and the client handles the operation as failed, but in fact, the file ends up on the server.

What makes me writing this BR is that in the client logfile I can find the following logging:

10-05 14:06:15:623 [ debug sync.propagator.upload.tus ] [ OCC::PropagateUploadFileTUS::startNextChunk ]:        Starting creation with upload: "/openmediavault_5.3.9-amd64.iso"
10-05 14:06:15:623 [ debug sync.propagator.upload.tus ] [ OCC::PropagateUploadFileTUS::makeCreationWithUploadJob ]:     FullPath: "/openmediavault_5.3.9-amd64.iso"
10-05 14:06:15:623 [ debug sync.propagator.upload.tus ] [ OCC::PropagateUploadFileTUS::startNextChunk ]:        Offset: 0 0 Chunk: 586153984 0
10-05 14:06:15:623 [ warning default ]: QObject::connect(QNetworkReply, OCC::UploadDevice): invalid nullptr parameter
10-05 14:06:15:623 [ warning default ]: QObject::connect(QNetworkReply, OCC::PropagateUploadFileTUS): invalid nullptr parameter
10-05 14:06:15:623 [ info sync.accessmanager ]: 4 "" "https://ocis.team.owncloud.works/remote.php/dav/files/kfreitag/" has X-Request-ID "26cd29ba-1c4a-4fce-41f3-c94a6709440a"
10-05 14:06:15:623 [ debug sync.cookiejar ]     [ OCC::CookieJar::cookiesForUrl ]:      QUrl("https://ocis.team.owncloud.works/remote.php/dav/files/kfreitag/") requests: ()

Note the strange warnings from QObject::connect()!

@TheOneRing TheOneRing added this to the 2.9.2 milestone Oct 5, 2021
TheOneRing added a commit that referenced this issue Feb 11, 2022
12-13 13:35:17:462 [ warning default ]: QObject::connect(QNetworkReply, OCC::UploadDevice): invalid null parameter
12-13 13:35:17:462 [ warning default ]: QObject::connect(QNetworkReply, OCC::PropagateUploadFileTUS): invalid null parameter

Fixes: #9121
@TheOneRing TheOneRing modified the milestones: 2.10, 2.10.1 Feb 11, 2022
TheOneRing added a commit that referenced this issue Feb 11, 2022
12-13 13:35:17:462 [ warning default ]: QObject::connect(QNetworkReply, OCC::UploadDevice): invalid null parameter
12-13 13:35:17:462 [ warning default ]: QObject::connect(QNetworkReply, OCC::PropagateUploadFileTUS): invalid null parameter

Fixes: #9121
TheOneRing added a commit that referenced this issue Feb 11, 2022
12-13 13:35:17:462 [ warning default ]: QObject::connect(QNetworkReply, OCC::UploadDevice): invalid null parameter
12-13 13:35:17:462 [ warning default ]: QObject::connect(QNetworkReply, OCC::PropagateUploadFileTUS): invalid null parameter

Fixes: #9121
@TheOneRing TheOneRing mentioned this issue Mar 10, 2022
55 tasks
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

2 participants