nextcloud / nextcloud/nextcloud-docker-dev

GS improvements

Open
#67 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
PHP
Stars
190
Forks
103
Avg merge
4h 36m
Merged PRs (30d)
1

Description

  • figure out how to make plain http work out of the box
  • add script to auto configure collabora
  • auto configure gss.master.csp-allow
  • trigger cron directly after install to trigger initial lookup server sync

Workaround patch for sharing with http

  • This makes sure that the protocol gets added to the lookup server entries
diff --git a/lib/private/User/User.php b/lib/private/User/User.php
index e7aa72fafb..17ef7412fa 100644
--- a/lib/private/User/User.php
+++ b/lib/private/User/User.php
@@ -552,7 +552,7 @@ class User implements IUser {
                if (strpos($url, 'https://') === 0) {
                        return substr($url, strlen('https://'));
                } elseif (strpos($url, 'http://') === 0) {
-                       return substr($url, strlen('http://'));
+                       // return substr($url, strlen('http://'));
                }

                return $url;

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Review the docker-compose development setup and installation/configuration entry points for the four checklist items, then inspect lib/private/User/User.php and its HTTP-sharing workaround. Completion means plain HTTP works out of the box, Collabora and gss.master.csp-allow are auto-configured, and the initial lookup server sync is triggered after installation.

Written by the indexing model from the issue text.

Assessment

Tech stack
docker-compose, php
Domain
devops, infrastructure
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.