Skip to content

fix(DAV): Adjust expected response code after nextcloud/server#54441 #837

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.

Sign up for GitHub

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

Jump to bottom
Merged
nickvergessen merged 2 commits into from
Dec 15, 2025
Merged

fix(DAV): Adjust expected response code after nextcloud/server#54441 #837

nickvergessen merged 2 commits into from
Dec 15, 2025

Conversation

@nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Dec 15, 2025 •
edited
Loading

  • Solves most failing tests
  • Permission PROPFIND is still broken features/sharing-user.feature:176 instead of multi-status it has 404 Not Found now.

@nickvergessen
fix(DAV): Adjust expected response code after nextcloud/server#54441
0062291 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(DAV): Propfind is blocked without read permissions now
535c567 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen force-pushed the bugfix/noid/adjust-expected-response-code branch from 83feda1 to 535c567 Compare December 15, 2025 10:53
@nickvergessen nickvergessen self-assigned this Dec 15, 2025
@nickvergessen
Copy link
Member Author

nickvergessen commented Dec 15, 2025

Checking psalm in #836 afterwards as it's not related to this PR here

@nickvergessen nickvergessen merged commit d329e6d into main Dec 15, 2025
42 of 43 checks passed
@nickvergessen nickvergessen deleted the bugfix/noid/adjust-expected-response-code branch December 15, 2025 11:17
@salmart-dev salmart-dev mentioned this pull request Dec 22, 2025
Merged
7 tasks
salmart-dev added a commit that referenced this pull request Jan 2, 2026
@salmart-dev
Revert "Merge pull request #837 from nextcloud/bugfix/noid/adjust-exp…
e4bf9fe 
…ected-response-code"

This reverts commit d329e6d, reversing
changes made to de09882.
salmart-dev added a commit that referenced this pull request Jan 2, 2026
@salmart-dev
Revert "Merge pull request #837 from nextcloud/bugfix/noid/adjust-exp…
0ef18e1 
…ected-response-code"

This reverts commit d329e6d, reversing
changes made to de09882.

Signed-off-by: Salvatore Martire <4652631+salmart-dev@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@salmart-dev salmart-dev salmart-dev approved these changes

Assignees

@nickvergessen nickvergessen

Labels

3. to review bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nickvergessen @salmart-dev