Skip to content

adapt test for #514#2255

Merged
ScharfViktor merged 1 commit into
mainfrom
adapt-test-for-#514
Feb 3, 2026
Merged

adapt test for #514#2255
ScharfViktor merged 1 commit into
mainfrom
adapt-test-for-#514

Conversation

@ScharfViktor
Copy link
Copy Markdown
Contributor

@ScharfViktor ScharfViktor commented Feb 3, 2026

@ScharfViktor
Copy link
Copy Markdown
Contributor Author

ScharfViktor commented Feb 3, 2026

@aduffeck We have problem in this step When user "Alice" moves file "textfile.txt" of space "myspace" to folder "docs" using its id in destination path

when we use destination with resourceId like "https://localhost:9200/remote.php/dav/spaces/4e02c0d4-54ae-4c7e-93f5-52a18e0dc816$85896fd2-3773-4192-b96b-b58b3dc96c9a!6b4d5b6f-8f7e-4154-89a6-8e4a60fa1c9f" we get always 400 error

how expected to work: owncloud/ocis#6739 (comment)

==> REQUEST [2026-02-03T9:54:38]
		MOVE /remote.php/dav/spaces/1bcde694-d09a-4674-969a-314409c13c52$240312b3-e585-4519-acf2-e1a4885851ff/textfile.txt
		X-Request-ID: apiSpacesDavOperation/moveByFileId.feature:663-666

		<== RESPONSE [2026-02-03T9:54:38]
		400 Bad Request
		X-Xss-Protection: 1; mode=block
		<== RES BODY
		<?xml version="1.0" encoding="UTF-8"?>
		<d:error xmlns:d="DAV" xmlns:s="[http://sabredav.org/ns"><s:exception>Sabre\DAV\Exception\BadRequest</s:exception><s:message>destination](http://sabredav.org/ns%22%3E%3Cs:exception%3ESabre/DAV/Exception/BadRequest%3C/s:exception%3E%3Cs:message%3Edestination) naming rules</s:message><s:errorcode></s:errorcode></d:error>

@aduffeck
Copy link
Copy Markdown
Member

aduffeck commented Feb 3, 2026

@ScharfViktor I added a commit to opencloud-eu/reva#514 which should fix that problem

@ScharfViktor ScharfViktor marked this pull request as ready for review February 3, 2026 14:15
@ScharfViktor ScharfViktor requested a review from aduffeck February 3, 2026 14:16
aduffeck
aduffeck reviewed Feb 3, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@aduffeck aduffeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm. What are the changes in docs and services about though?

@ScharfViktor ScharfViktor requested a review from aduffeck February 3, 2026 14:47
@ScharfViktor
Copy link
Copy Markdown
Contributor Author

ScharfViktor commented Feb 3, 2026

lgtm. What are the changes in docs and services about though?

deleted this changes

aduffeck
aduffeck approved these changes Feb 3, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@aduffeck aduffeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would change the commit message, other than that it lgtm.

@ScharfViktor ScharfViktor merged commit 400dc9f into main Feb 3, 2026
59 checks passed
@ScharfViktor ScharfViktor deleted the adapt-test-for-#514 branch February 3, 2026 18:51
@openclouders openclouders mentioned this pull request Feb 3, 2026
Merged
1 task
rhafer added a commit to rhafer/opencloud that referenced this pull request Feb 5, 2026
@rhafer
Revert "adapt test for opencloud-eu#514 (opencloud-eu#2255)"
55f4051 
This reverts commit 400dc9f.
rhafer added a commit that referenced this pull request Feb 5, 2026
@rhafer
Revert "adapt test for #514 (#2255)"
f21207e 
This reverts commit 400dc9f.
renovate Bot added a commit to sdwilsh/ansible-playbooks that referenced this pull request Feb 9, 2026
@renovate
chore(deps): update opencloudeu/opencloud-rolling docker tag to v5.0.2
41f9708 
##### [\`5.0.2\`](https://github.com/opencloud-eu/opencloud/blob/HEAD/CHANGELOG.md#502---2026-02-05)

##### ❤️ Thanks to all contributors! ❤️

[@AlexAndBear](https://github.com/AlexAndBear), [@ScharfViktor](https://github.com/ScharfViktor), [@flimmy](https://github.com/flimmy), [@individual-it](https://github.com/individual-it), [@rhafer](https://github.com/rhafer), [@saw-jan](https://github.com/saw-jan)

##### 🐛 Bug Fixes

- \[full-ci] reva-bump-2.42.3 \[[#2276](opencloud-eu/opencloud#2276)]

##### ✅ Tests

- adapt test for [#514](opencloud-eu/opencloud#514) \[[#2255](opencloud-eu/opencloud#2255)]
- api-test: upload-rename-download file with back slash \[[#2239](opencloud-eu/opencloud#2239)]
- \[full-ci]\[tests-only] test: add hook failures to the test failures list \[[#2041](opencloud-eu/opencloud#2041)]

##### 📚 Documentation

- docs(proxy): Clarify PROXY\_OIDC\_USERINFO\_CACHE\_TTL value \[[#2256](opencloud-eu/opencloud#2256)]

##### 📦️ Dependencies

- \[full-ci] reva-bump-2.42.2 \[[#2270](opencloud-eu/opencloud#2270)]
- build(deps): bump github.com/grpc-ecosystem/grpc-gateway/v2 from 2.27.5 to 2.27.6 \[[#2238](opencloud-eu/opencloud#2238)]
sdwilsh pushed a commit to sdwilsh/ansible-playbooks that referenced this pull request Feb 9, 2026
@renovate @sdwilsh
chore(deps): update opencloudeu/opencloud-rolling docker tag to v5.0.2
59760a0 
##### [\`5.0.2\`](https://github.com/opencloud-eu/opencloud/blob/HEAD/CHANGELOG.md#502---2026-02-05)

##### ❤️ Thanks to all contributors! ❤️

[@AlexAndBear](https://github.com/AlexAndBear), [@ScharfViktor](https://github.com/ScharfViktor), [@flimmy](https://github.com/flimmy), [@individual-it](https://github.com/individual-it), [@rhafer](https://github.com/rhafer), [@saw-jan](https://github.com/saw-jan)

##### 🐛 Bug Fixes

- \[full-ci] reva-bump-2.42.3 \[[#2276](opencloud-eu/opencloud#2276)]

##### ✅ Tests

- adapt test for [#514](opencloud-eu/opencloud#514) \[[#2255](opencloud-eu/opencloud#2255)]
- api-test: upload-rename-download file with back slash \[[#2239](opencloud-eu/opencloud#2239)]
- \[full-ci]\[tests-only] test: add hook failures to the test failures list \[[#2041](opencloud-eu/opencloud#2041)]

##### 📚 Documentation

- docs(proxy): Clarify PROXY\_OIDC\_USERINFO\_CACHE\_TTL value \[[#2256](opencloud-eu/opencloud#2256)]

##### 📦️ Dependencies

- \[full-ci] reva-bump-2.42.2 \[[#2270](opencloud-eu/opencloud#2270)]
- build(deps): bump github.com/grpc-ecosystem/grpc-gateway/v2 from 2.27.5 to 2.27.6 \[[#2238](opencloud-eu/opencloud#2238)]
rhafer added a commit to rhafer/opencloud that referenced this pull request Feb 9, 2026
@rhafer
Reapply "adapt test for opencloud-eu#514 (opencloud-eu#2255)"
8b446a6 
This reverts commit f21207e.
@rhafer rhafer mentioned this pull request Feb 9, 2026
Merged
ScharfViktor pushed a commit that referenced this pull request Feb 9, 2026
@rhafer
Reapply "adapt test for #514 (#2255)" (#2305)
c33850f 
This reverts commit f21207e.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aduffeck aduffeck aduffeck approved these changes

Assignees

No one assigned

Labels

Type:Test

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ScharfViktor @aduffeck