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]: Webdav availability request gives malformed response #44205

Closed
4 of 8 tasks
GVodyanov opened this issue Mar 14, 2024 · 5 comments · Fixed by #44344
Closed
4 of 8 tasks

[Bug]: Webdav availability request gives malformed response #44205

GVodyanov opened this issue Mar 14, 2024 · 5 comments · Fixed by #44344

Comments

@GVodyanov
Copy link
Contributor

⚠️ This issue respects the following points: ⚠️

Bug description

Webdav availability request gives malformed response, especially when a user doesn't have any availability slots configured so far

Steps to reproduce

  1. Create a new user
  2. Go to availability settings
  3. See "Failed to load availability" (first issue)
  4. Set some availability slots and save
  5. Reload the page
  6. The availability slots aren't shown

Expected behavior

No errors and slots are saved

Installation method

Other Community project

Nextcloud Server version

master

Operating system

Other

PHP engine version

None

Web server

None

Database engine version

None

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

Fresh Nextcloud Server install

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

No response

List of activated Apps

No response

Nextcloud Signing status

No response

Nextcloud Logs

No response

Additional info

No response

@GVodyanov GVodyanov added bug 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Mar 14, 2024
@GVodyanov GVodyanov added 1. to develop Accepted and waiting to be taken care of and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Mar 14, 2024
@ChristophWurst ChristophWurst added this to the Nextcloud 29 milestone Mar 14, 2024
@GVodyanov GVodyanov removed their assignment Mar 18, 2024
@st3iny st3iny moved this from 📄 To do to 🏗️ In progress in 💌 📅 👥 Groupware team Mar 19, 2024
@st3iny st3iny assigned st3iny and unassigned ChristophWurst Mar 19, 2024
@st3iny st3iny added 2. developing Work in progress and removed 1. to develop Accepted and waiting to be taken care of labels Mar 19, 2024
@st3iny
Copy link
Member

st3iny commented Mar 20, 2024

This is caused by webdav switching their http client library from axios to "native" fetch1.

The property response.data is not available any more as it was populated by axios. Instead, we have to manually await the body now, e.g. await response.text().

I wonder if there are more instances of this across our code base.

Footnotes

  1. https://github.com/perry-mitchell/webdav-client?tab=readme-ov-file#requests

@st3iny
Copy link
Member

st3iny commented Mar 20, 2024

I did not find another broken usage inside the server repository.

rg customRequest apps/*/src dev_apps/*/src

@ChristophWurst
Copy link
Member

This is caused by webdav switching their http client library from axios to "native" fetch1.

https://github.com/perry-mitchell/webdav-client?tab=readme-ov-file#customrequest oh no

@st3iny
Copy link
Member

st3iny commented Mar 20, 2024

Yeah, they should definitely adjust their readme 😅

@st3iny
Copy link
Member

st3iny commented Mar 20, 2024

Yeah, they should definitely adjust their readme 😅

perry-mitchell/webdav-client#371

@ChristophWurst ChristophWurst added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Mar 20, 2024
@github-project-automation github-project-automation bot moved this from 🏗️ In progress to ☑️ Done in 💌 📅 👥 Groupware team Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

4 participants