-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[Bug]: PROPFIND on dav returns null instead of 404 #32514
Comments
without looking at the code, one change I'm aware of is that we touched a lot of code to make it ready for PHP 8.1 we also touched a bit of DAV for metadata @CarlSchwan |
This also happens for rich-workspaces: Now on master/NC24 it seems that null is returned until user really creates a file / types something. |
@CarlSchwan please have a look |
we forgot to backport #32548 😢 |
$32548 does not fix this. |
This is a regression from nextcloud/text@3e40b7b @max-nextcloud |
Regression tests for nextcloud/server#32514 (comment) Signed-off-by: Max <max@nextcloud.com>
Regression tests for nextcloud/server#32514 (comment) Signed-off-by: Max <max@nextcloud.com>
Regression tests for nextcloud/server#32514 (comment) Signed-off-by: Max <max@nextcloud.com>
Regression tests for nextcloud/server#32514 (comment) Signed-off-by: Max <max@nextcloud.com>
Regression tests for nextcloud/server#32514 (comment) Signed-off-by: Max <max@nextcloud.com>
Regression tests for nextcloud/server#32514 (comment) Signed-off-by: Max <max@nextcloud.com>
Regression tests for nextcloud/server#32514 (comment) Signed-off-by: Max <max@nextcloud.com>
Regression tests for nextcloud/server#32514 (comment) Signed-off-by: Max <max@nextcloud.com>
Regression tests for nextcloud/server#32514 (comment) Signed-off-by: Max <max@nextcloud.com>
Regression tests for nextcloud/server#32514 (comment) Signed-off-by: Max <max@nextcloud.com>
Regression tests for nextcloud/server#32514 (comment) Signed-off-by: Max <max@nextcloud.com>
Bug description
With NC23 this works:
Now with master it is in 200 status part:
oc:comments-unread</oc:comments-unread>
As Android expects this to be an integer, it is translated to null, which fails.
Steps to reproduce
Expected behavior
Same as in NC23: oc:comments-unread/ in 404 status part.
Installation method
Manual installation
Operating system
No response
PHP engine version
No response
Web server
No response
Database engine version
No response
Is this bug present after an update or on a fresh install?
No response
Are you using the Nextcloud Server Encryption module?
No response
What user-backends are you using?
Configuration report
No response
List of activated Apps
Nextcloud Signing status
Nextcloud Logs
Additional info
No response
The text was updated successfully, but these errors were encountered: