-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Check quota on file copy #35009
Check quota on file copy #35009
Conversation
/backport to stable25 |
/backport to stable24 |
/backport to stable23 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
psalm not happy and an integration test fails now due to beforeCopy and a dot file that is not allowed. I suspect that the latter is more about a test case doing something dodgy than a bug |
Seems like https://drone.nextcloud.com/nextcloud/server/25500/20/3 fails. Not sure if related though... |
{
"Exception": "OCA\\DAV\\Connector\\Sabre\\Exception\\InvalidPath",
"Message": "Dot files are not allowed",
"Code": 0,
"Trace": [
{
"file": "/drone/src/apps/dav/lib/Connector/Sabre/QuotaPlugin.php",
"line": 158,
"function": "getNodeForPath",
"class": "OCA\\DAV\\Connector\\Sabre\\ObjectTree",
"type": "->"
},
{
"file": "/drone/src/3rdparty/sabre/event/lib/WildcardEmitterTrait.php",
"line": 89,
"function": "beforeCopy",
"class": "OCA\\DAV\\Connector\\Sabre\\QuotaPlugin",
"type": "->"
},
{
"file": "/drone/src/3rdparty/sabre/dav/lib/DAV/CorePlugin.php",
"line": 648,
"function": "emit",
"class": "Sabre\\DAV\\Server",
"type": "->"
},
{
"file": "/drone/src/3rdparty/sabre/event/lib/WildcardEmitterTrait.php",
"line": 89,
"function": "httpCopy",
"class": "Sabre\\DAV\\CorePlugin",
"type": "->"
},
{
"file": "/drone/src/3rdparty/sabre/dav/lib/DAV/Server.php",
"line": 472,
"function": "emit",
"class": "Sabre\\DAV\\Server",
"type": "->"
},
{
"file": "/drone/src/3rdparty/sabre/dav/lib/DAV/Server.php",
"line": 253,
"function": "invokeMethod",
"class": "Sabre\\DAV\\Server",
"type": "->"
},
{
"file": "/drone/src/3rdparty/sabre/dav/lib/DAV/Server.php",
"line": 321,
"function": "start",
"class": "Sabre\\DAV\\Server",
"type": "->"
},
{
"file": "/drone/src/apps/dav/appinfo/v1/webdav.php",
"line": 85,
"function": "exec",
"class": "Sabre\\DAV\\Server",
"type": "->"
},
{
"file": "/drone/src/remote.php",
"line": 171,
"args": [
"/drone/src/apps/dav/appinfo/v1/webdav.php"
],
"function": "require_once"
}
],
"File": "/drone/src/apps/dav/lib/Connector/Sabre/ObjectTree.php",
"Line": 124,
"message": "Dot files are not allowed",
"exception": {},
"CustomMessage": "Dot files are not allowed"
} Sounds very related |
|
Nice catch, thank you. Then the same change is needed in beforeMove, no? |
/rebase |
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Robin Appelman <robin@icewind.nl>
Let’s be safe Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
3fadfc6
to
feafa63
Compare
/backport to stable24 |
/backport to stable23 |
The backport to stable24 failed. Please do this backport manually. |
The backport to stable23 failed. Please do this backport manually. |
Fixes #34975
Signed-off-by: Côme Chilliet come.chilliet@nextcloud.com