From 10689721c7c15eb5e26b1af7b92ea0693d7763ab Mon Sep 17 00:00:00 2001
From: Sergey Sharybin <sergey@blender.org>
Date: Wed, 13 Sep 2023 11:11:28 +0200
Subject: [PATCH] Move `Require valid-user` to global part of configuration

By default it requires valid-user for the entirety of the /repo/
folder, which matches the old behavior.

Doing so in the global configuration allows to more easily apply
per-site configuration tweaks like allowing anonymous read-only
access for internal IP range.
---
 README.md               | 4 ++++
 docker/000-default.conf | 4 ++++
 svnman/create_repo.go   | 1 -
 3 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 2e8194c..15fef7e 100644
--- a/README.md
+++ b/README.md
@@ -75,6 +75,10 @@ subpackage.
     IncludeOptional svn/*.conf
     IncludeOptional svn/*/*.conf
 
+    <Location /repo/>
+      Require valid-user
+    </Location>
+
     <Location /api/>
         AuthType Basic
         AuthName "Blender Cloud SVN Manager API"
diff --git a/docker/000-default.conf b/docker/000-default.conf
index d0883d5..474d5d7 100644
--- a/docker/000-default.conf
+++ b/docker/000-default.conf
@@ -12,6 +12,10 @@
     IncludeOptional svn/*.conf
     IncludeOptional svn/*/*.conf
 
+    <Location /repo/>
+      Require valid-user
+    </Location>
+
     <Location /api/>
         AuthType Basic
         AuthName "Blender Cloud SVN Manager API"
diff --git a/svnman/create_repo.go b/svnman/create_repo.go
index 82d132e..10b44d5 100644
--- a/svnman/create_repo.go
+++ b/svnman/create_repo.go
@@ -20,7 +20,6 @@ const apacheTemplate = `# Location directive for project %q
     AuthType Basic
     AuthName %q
     AuthUserFile %s
-    Require valid-user
 </Location>
 `
 
-- 
2.30.2