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

[Bug]: dav search request responds with HTTP/1.1 500 Internal Server Error #49972

Open
4 of 8 tasks
wlklch opened this issue Dec 26, 2024 · 2 comments · May be fixed by #51592
Open
4 of 8 tasks

[Bug]: dav search request responds with HTTP/1.1 500 Internal Server Error #49972

wlklch opened this issue Dec 26, 2024 · 2 comments · May be fixed by #51592
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 30-feedback bug

Comments

@wlklch
Copy link

wlklch commented Dec 26, 2024

⚠️ This issue respects the following points: ⚠️

Bug description

a SEARCH request to .../remote.php/dav/uri ... for a bad fileid gives in consistent reponses:

<d:searchrequest xmlns: ......
<d:where>
<d:eq><d:prop>oc:fileid/</d:prop><d:literal>q234</d:literal></d:eq>

responds with HTTP/1.1 500 Internal Server Error and I could not find any indication in the response header, that it is simply bad input data

searching for fileid=234q or fileid=234, however, both correctly respondswith HTTP/1.1 207 Multi-Status and the body an empty xml-element <d:multistatus .../>

Steps to reproduce

  1. see above

Expected behavior

multistatuis as described above

Nextcloud Server version

30

Operating system

None

PHP engine version

None

Web server

None

Database engine version

None

Is this bug present after an update or on a fresh install?

None

Are you using the Nextcloud Server Encryption module?

None

What user-backends are you using?

  • Default user-backend (database)
  • LDAP/ Active Directory
  • SSO - SAML
  • Other

Configuration report

List of activated Apps

Nextcloud Signing status

Nextcloud Logs

Additional info

the server is my hosting provider, so I do not know any details, to provide them needs some work. However I guess, the details are irrelevant.

@wlklch wlklch added 0. Needs triage Pending check for reproducibility or if it fits our roadmap bug labels Dec 26, 2024
@SystemKeeper
Copy link
Contributor

Wondering if this is also related to #49960 maybe?

@wlklch
Copy link
Author

wlklch commented Dec 27, 2024

susnux added a commit that referenced this issue Mar 19, 2025
* Resolves #49972

Currently a TypeError is thrown when casting fails,
this lead to a HTTP 500 error. Instead throw a proper
InvalidArgumentError so the user receives a HTTP 400.

Signed-off-by: Ferdinand Thiessen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 30-feedback bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants