[SAC] uncommitted changes in TracSVN apache config

Committees SAC mailing list
1

Working on an apache configuration change for GEOS [1]
I found many changes in Apache configuration of TracSVN machine
that were not committed to git.

Some of those changes seem related to system upgrade
(Martin Spott could you please commit them ?)
but some other seem specific to GRASS changes
(Markus Neteler could you please commit them ?)

Full list of changes:

# modified: apache2.conf
# modified: conf.d/security
# modified: envvars
# deleted: httpd.conf
# modified: includes/subversion/grass.conf
# modified: mods-available/alias.conf
# modified: mods-available/dav_svn.conf
# modified: mods-available/dav_svn.load
# modified: mods-available/disk_cache.conf
# modified: mods-available/ldap.conf
# modified: mods-available/mime.conf
# modified: mods-available/mod-evasive.conf
# modified: mods-available/ssl.conf
# deleted: mods-available/version.load
# modified: sites-available/openlayers.conf
# modified: sites-available/subversion.conf
# modified: sites-available/trac.conf

[1] https://trac.osgeo.org/geos/ticket/934

--

  () Free GIS & Flash consultant/developer
  /\ https://strk.kbt.io/services.html

2

On Tue, Oct 9, 2018 at 12:13 AM Sandro Santilli <strk@kbt.io> wrote:

Working on an apache configuration change for GEOS [1]
I found many changes in Apache configuration of TracSVN machine
that were not committed to git.

Some of those changes seem related to system upgrade
(Martin Spott could you please commit them ?)
but some other seem specific to GRASS changes
(Markus Neteler could you please commit them ?)

diff --git a/includes/subversion/grass.conf b/includes/subversion/grass.conf
index 163ade7..a4bc4f0 100644
--- a/includes/subversion/grass.conf
+++ b/includes/subversion/grass.conf
@@ -1,12 +1,12 @@
-
-
     <Location /grass>
       DAV svn
       SVNPath /var/www/svn/repos/grass
       AuthzSVNAccessFile /var/www/svn/repos/grass/authz

- Include includes/ldap_auth_url.inc

+# <LimitExcept GET PROPFIND OPTIONS REPORT>
       Satisfy Any
       Require valid-user
+ Include includes/ldap_auth_url.inc
+# </LimitExcept>
     </Location>

I have no idea who and why that was changed (will be ok of course!).
Hence I leave to to whom made the modification to commit it.

Markus

Full list of changes:

# modified: apache2.conf
# modified: conf.d/security
# modified: envvars
# deleted: httpd.conf
# modified: includes/subversion/grass.conf
# modified: mods-available/alias.conf
# modified: mods-available/dav_svn.conf
# modified: mods-available/dav_svn.load
# modified: mods-available/disk_cache.conf
# modified: mods-available/ldap.conf
# modified: mods-available/mime.conf
# modified: mods-available/mod-evasive.conf
# modified: mods-available/ssl.conf
# deleted: mods-available/version.load
# modified: sites-available/openlayers.conf
# modified: sites-available/subversion.conf
# modified: sites-available/trac.conf

[1] https://trac.osgeo.org/geos/ticket/934

--

  () Free GIS & Flash consultant/developer
  /\ https://strk.kbt.io/services.html

3

On Tue, Oct 09, 2018 at 10:28:56AM +0200, Markus Neteler wrote:

On Tue, Oct 9, 2018 at 12:13 AM Sandro Santilli <strk@kbt.io> wrote:
>
> Working on an apache configuration change for GEOS [1]
> I found many changes in Apache configuration of TracSVN machine
> that were not committed to git.
>
> Some of those changes seem related to system upgrade
> (Martin Spott could you please commit them ?)
> but some other seem specific to GRASS changes
> (Markus Neteler could you please commit them ?)

diff --git a/includes/subversion/grass.conf b/includes/subversion/grass.conf
index 163ade7..a4bc4f0 100644
--- a/includes/subversion/grass.conf
+++ b/includes/subversion/grass.conf
@@ -1,12 +1,12 @@
-
-
     <Location /grass>
       DAV svn
       SVNPath /var/www/svn/repos/grass
       AuthzSVNAccessFile /var/www/svn/repos/grass/authz

- Include includes/ldap_auth_url.inc

+# <LimitExcept GET PROPFIND OPTIONS REPORT>
       Satisfy Any
       Require valid-user
+ Include includes/ldap_auth_url.inc
+# </LimitExcept>
     </Location>

I have no idea who and why that was changed (will be ok of course!).
Hence I leave to to whom made the modification to commit it.

The change seems to make GRASS SVN access require a valid OSGeo
account even for only reading it. Is this what you want ?

--strk;

4

On Tue, Oct 9, 2018 at 12:53 PM Sandro Santilli <strk@kbt.io> wrote:

On Tue, Oct 09, 2018 at 10:28:56AM +0200, Markus Neteler wrote:
> On Tue, Oct 9, 2018 at 12:13 AM Sandro Santilli <strk@kbt.io> wrote:
> >
> > Working on an apache configuration change for GEOS [1]
> > I found many changes in Apache configuration of TracSVN machine
> > that were not committed to git.
> >
> > Some of those changes seem related to system upgrade
> > (Martin Spott could you please commit them ?)
> > but some other seem specific to GRASS changes
> > (Markus Neteler could you please commit them ?)
>
>
> diff --git a/includes/subversion/grass.conf b/includes/subversion/grass.conf
> index 163ade7..a4bc4f0 100644
> --- a/includes/subversion/grass.conf
> +++ b/includes/subversion/grass.conf
> @@ -1,12 +1,12 @@
> -
> -
> <Location /grass>
> DAV svn
> SVNPath /var/www/svn/repos/grass
> AuthzSVNAccessFile /var/www/svn/repos/grass/authz
>
> - Include includes/ldap_auth_url.inc
>
> +# <LimitExcept GET PROPFIND OPTIONS REPORT>
> Satisfy Any
> Require valid-user
> + Include includes/ldap_auth_url.inc
> +# </LimitExcept>
> </Location>
>
> I have no idea who and why that was changed (will be ok of course!).
> Hence I leave to to whom made the modification to commit it.

The change seems to make GRASS SVN access require a valid OSGeo
account even for only reading it. Is this what you want ?

Since I didn't change the GRASS SVN config, I do not have the answer.
Adding Martin L and Vaclav in CC, perhaps they know?

Markus