Skip to content

Only try to limit search to spaces if there's a space id to limit to#2834

Open
aduffeck wants to merge 4 commits into
opencloud-eu:mainfrom
aduffeck:fix-space-search
Open

Only try to limit search to spaces if there's a space id to limit to#2834
aduffeck wants to merge 4 commits into
opencloud-eu:mainfrom
aduffeck:fix-space-search

Conversation

@aduffeck
Copy link
Copy Markdown
Member

This enables clients to drop the /remote.php prefix of search requests.

@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented May 26, 2026

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 4 complexity · 0 duplication

Metric Results
Complexity 4
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@ScharfViktor
Copy link
Copy Markdown
Contributor

we run tests with remote.php by default now. lets run it again after merging #2826

@aduffeck aduffeck force-pushed the fix-space-search branch from 4e4508b to 6bb2324 Compare May 26, 2026 11:43
@ScharfViktor
Copy link
Copy Markdown
Contributor

found why apiSearch1/search.feature:176 is failed

if we send REPORT https://localhost:9200/dav/spaces -> shouldn't we get <d:href>/remote.php/dav/spaces/a26d06e4-d45e-4878-8b0a-26125d9046db$3eec4f61-ee6c-4e78-8c52-734fe106c5af</d:href> (with remore.php)?

Screenshot 2026-05-26 at 16 49 53

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants