Skip to content

Fix #71: Server Path Prefix#150

Merged
i2h3 merged 1 commit intodevelopfrom
71-server-path-prefix
Jul 14, 2025
Merged

Fix #71: Server Path Prefix#150
i2h3 merged 1 commit intodevelopfrom
71-server-path-prefix

Conversation

@i2h3
Copy link
Collaborator

@i2h3 i2h3 commented Jul 2, 2025

@i2h3 i2h3 added this to the 4.5.0 milestone Jul 2, 2025
@i2h3 i2h3 requested review from marinofaggiana and mpivchev July 2, 2025 14:32
@i2h3 i2h3 self-assigned this Jul 2, 2025
@i2h3 i2h3 added the bug Something isn't working label Jul 2, 2025
@i2h3 i2h3 force-pushed the 71-server-path-prefix branch from a369886 to 6d3ffce Compare July 2, 2025 14:40
@i2h3 i2h3 marked this pull request as ready for review July 2, 2025 14:40
@i2h3 i2h3 force-pushed the 71-server-path-prefix branch from 6d3ffce to d9b26d1 Compare July 3, 2025 06:12
@i2h3 i2h3 force-pushed the 71-server-path-prefix branch from d9b26d1 to 5e12b95 Compare July 3, 2025 10:58
- Refactored network router code.
- NoteRouter.swift got split up into its individual types.
- OCSRouter and StatusRouter now also take the server path prefix into consideration.
- Removed dead code in NotesTableViewController due to new login introduced recently.

Signed-off-by: Iva Horn <iva.horn@icloud.com>
@i2h3 i2h3 force-pushed the 71-server-path-prefix branch from 5e12b95 to 2ddda14 Compare July 3, 2025 11:02
@mpivchev
Copy link
Collaborator

mpivchev commented Jul 3, 2025

What should I test for this?

@i2h3
Copy link
Collaborator Author

i2h3 commented Jul 3, 2025

@mpivchev Whether the app works with Nextcloud deployments on the top level of a domain (in example https://example.org) but also when Nextcloud's root is in a subdirectory of a domain (in example https://example.org/nextcloud).

@i2h3 i2h3 merged commit b9996e5 into develop Jul 14, 2025
1 check passed
@i2h3 i2h3 deleted the 71-server-path-prefix branch July 14, 2025 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GET /ocs/v1.php/cloud/capabilities misses the web root, i.e. /nextcloud

2 participants

Comments