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

OCM. federated user gets a server crash when uploading files via tus #10285

Closed
ScharfViktor opened this issue Oct 11, 2024 · 1 comment
Closed
Assignees
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug

Comments

@ScharfViktor
Copy link
Contributor

ScharfViktor commented Oct 11, 2024

Web UI 10.3.0
Ocis: 6.5.0

Steps:
Given using server "LOCAL"
And "Alice" has generated invitation
And using server "REMOTE"
And "Brian" has accepted invitation
And using server "LOCAL"
And "Alice" shares folder f1 to "Brian"
When "Brian" tries to upload file to shared folder f1 via TUS

Actual:
server crash:

panic: runtime error: index out of range [0] with length 0
2024-10-11 09:29:30 
2024-10-11 09:29:30 goroutine 602 [running]:
2024-10-11 09:29:30 github.com/cs3org/reva/v2/pkg/rhttp/datatx/manager/tus.(*manager).Handler.func1()
2024-10-11 09:29:30     github.com/cs3org/reva/[email protected]/pkg/rhttp/datatx/manager/tus/tus.go:139 +0x31c
2024-10-11 09:29:30 created by github.com/cs3org/reva/v2/pkg/rhttp/datatx/manager/tus.(*manager).Handler in goroutine 577
2024-10-11 09:29:30     github.com/cs3org/reva/[email protected]/pkg/rhttp/datatx/manager/tus/tus.go:130 +0x354
2024-10-11 09:29:31 2024/10/11 07:29:31 INFO memory is not limited, skipping package=github.com/KimMachineGun/automemlimit/memlimit
2024-10-11 09:29:31 2024/10/11 07:29:31 Could not create config: config file already exists, use --force-overwrite to overwrite or --diff to show diff
2024-10-11 09:29:32 2024/10/11 07:29:32 INFO memory is not limited, skipping package=github.com/KimMachineGun/automemlimit/memlimit

Alice can see uploaded file. After restarting Brian instance -> file is exist
Expected: no server crash

CC @2403905

@micbar
Copy link
Contributor

micbar commented Oct 17, 2024

@2403905 Done? Please close then 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug
Projects
Status: Done
Development

No branches or pull requests

3 participants