Opened 6 years ago

Closed 6 years ago

Last modified 10 months ago

#934 closed task (fixed)

disable svn access from the web

Reported by: gdt Owned by: geos-devel@…
Priority: minor Milestone: Management
Component: Default Version: main
Severity: Annoyance Keywords:
Cc:

Description

The subversion repository has been superceded by git, but it is still accessible. So people with existing svn checkouts who didn't notice or remember the git transition who do "svn up" and expect to be on master can be misled.

The desired property is that "svn up" (or "svn co") will fail, or perhaps will result in a checkout that is obviously not usable.

It seems useful to have the svn source browser in trac still work, in case there are source links in tickets. But that is not in conflict with svn from the internet being denied, perhaps by an acl.

(leaving on Default because there isn't an appropriate component)

Change History (5)

comment:1 by strk, 6 years ago

Resolution: fixed
Status: newclosed

SVN was made read-only but still accessible as per #934 on April 5th 2017. Now I turned it completely inaccessible (forbidden) with custom error response pointing to https://trac.osgeo.org/geos/wiki/CodeRepository Unfortunately it doesn't look like git-svn shows the custom response, didn't test direct SVN client

comment:2 by robe, 6 years ago

Using my svn windows repo browser, I see a message

Access /geos forbidden

When I do

svn co https://svn.osgeo.org/geos/branches/3.6


I get

svn: E175013: Unable to connect to a repository at URL 'https://svn.osgeo.org/geos/branches/3.6'
svn: E175013: Access to '/geos/branches/3.6' forbidden

So I think it's good enough

comment:3 by strk, 6 years ago

FYI: I've now also made the SVN repository "hidden" from the Trac browser menu (references to revisions from tickets still work)

comment:4 by Paul Ramsey <pramsey@…>, 10 months ago

In e938543/git:

Explicitly set endianness for some tests so that output matches expected. Closes #934

comment:5 by Paul Ramsey <pramsey@…>, 10 months ago

In 6bc91ff/git:

Explicitly set endianness for some tests so that output matches expected. Closes #934

Note: See TracTickets for help on using tickets.