Skip to content

Fix public uploads for nextcloud subdirectories#1619

Open
gouverneurp wants to merge 3 commits into
pulsejet:masterfrom
gouverneurp:fix-public-uploads-for-nextcloud-subdirectories
Open

Fix public uploads for nextcloud subdirectories#1619
gouverneurp wants to merge 3 commits into
pulsejet:masterfrom
gouverneurp:fix-public-uploads-for-nextcloud-subdirectories

Conversation

@gouverneurp

Copy link
Copy Markdown
Contributor

This PR fixes uploads to public folders (for unregistered users) in Nextcloud instances that are deployed in a subdirectory.

For example, when a Nextcloud instance is hosted at www.example-cloud.com/nextcloud instead of directly at www.example-cloud.com, the previous implementation did not work correctly.

gouverneurp and others added 2 commits February 10, 2026 21:23
Signed-off-by: Philip Gouverneur <philipgouverneur@gmx.de>
@gouverneurp

Copy link
Copy Markdown
Contributor Author

@major-mayer, would you be interested to test this again?
Your help was much appreciated in the past!

Signed-off-by: Philip Gouverneur <philipgouverneur@gmx.de>
@major-mayer

Copy link
Copy Markdown

Hi @gouverneurp ,
I am glad to hear that 😊
I could give it a try, but I guess I need a Nextcloud environment that is hosted in a subfolder, right?
My dev environment is usually just accessible via http://localhost
Probably will take a while for me

@gouverneurp

gouverneurp commented Feb 10, 2026

Copy link
Copy Markdown
Contributor Author

Thank you for your help!
It would be fine for me if you tested it without a subdirectory, so you can also confirm that this isn’t breaking anything.
But in the end, @pulsejet needs to decide.

@major-mayer

Copy link
Copy Markdown

All right, I just tested this branch without a subdirectory hosting and the upload for external shares is still working flawlessly.
I have no objections against this PR :)

@mimedo

mimedo commented Mar 16, 2026

Copy link
Copy Markdown

Does this PR fixes #1634 as well?

@poggenpower

Copy link
Copy Markdown

@gouverneurp thank you for this fix. I tapped into the same issue.
I have created a patch from this PR and applied it to version 8.0.1 of memories and it fixed the issues.
I really would appreciate if this get merged and released.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants