# [PATCH] http-push: Make WebDAV work with (broken?) default apache2 WebDAV module 3 messages from 2006-08-09 to 2006-08-11. Participants: Rutger Nijlunsing, Junio C Hamano, Nick Hengeveld. Thread: https://gitlist.dev/t/5186 ## Rutger Nijlunsing, 2006-08-09 18:54 Subject: [PATCH] http-push: Make WebDAV work with (broken?) default apache2 WebDAV module Message-ID: <20060809185423.GB7841@nospam.com> URL: https://gitlist.dev/e/20060809185423.GB7841%40nospam.com ``` WebDAV on Debian unstable cannot handle renames on WebDAV from file.ext to newfile (without ext) when newfile* already exists. Normally, git creates a file like 'objects/xx/sha1.token', which is renamed to 'objects/xx/sha1' when transferred completely. Just use '_' instead of '.' so WebDAV doesn't see it as an extension change. Signed-off-by: Rutger Nijlunsing Acked-by: Johannes Schindelin --- http-push.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/http-push.c b/http-push.c index 4021e7d..d45733e 100644 --- a/http-push.c +++ b/http-push.c @@ -530,7 +530,7 @@ static void start_put(struct transfer_re request->dest = xmalloc(strlen(request->url) + 14); sprintf(request->dest, "Destination: %s", request->url); posn += 38; - *(posn++) = '.'; + *(posn++) = '_'; strcpy(posn, request->lock->token); slot = get_active_slot(); -- 1.4.2.rc2.g7cb1 -- Rutger Nijlunsing ---------------------------------- eludias ed dse.nl never attribute to a conspiracy which can be explained by incompetence ---------------------------------------------------------------------- ``` ## Junio C Hamano, 2006-08-09 19:33 Subject: Re: [PATCH] http-push: Make WebDAV work with (broken?) default apache2 WebDAV module Message-ID: <7vk65h3drd.fsf@assigned-by-dhcp.cox.net> URL: https://gitlist.dev/e/7vk65h3drd.fsf%40assigned-by-dhcp.cox.net In-Reply-To: <20060809185423.GB7841@nospam.com> ``` Rutger Nijlunsing writes: > WebDAV on Debian unstable cannot handle renames on WebDAV from > file.ext to newfile (without ext) when newfile* already > exists. Normally, git creates a file like 'objects/xx/sha1.token', > which is renamed to 'objects/xx/sha1' when transferred completely. > > Just use '_' instead of '.' so WebDAV doesn't see it as an extension > change. > > Signed-off-by: Rutger Nijlunsing > Acked-by: Johannes Schindelin Good problem and workaround description. Nick, does this look sane to you, or is there a more kosher way to handle this? > --- > http-push.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/http-push.c b/http-push.c > index 4021e7d..d45733e 100644 > --- a/http-push.c > +++ b/http-push.c > @@ -530,7 +530,7 @@ static void start_put(struct transfer_re > request->dest = xmalloc(strlen(request->url) + 14); > sprintf(request->dest, "Destination: %s", request->url); > posn += 38; > - *(posn++) = '.'; > + *(posn++) = '_'; > strcpy(posn, request->lock->token); > > slot = get_active_slot(); > -- > 1.4.2.rc2.g7cb1 > > > -- > Rutger Nijlunsing ---------------------------------- eludias ed dse.nl > never attribute to a conspiracy which can be explained by incompetence > ---------------------------------------------------------------------- ``` ## Nick Hengeveld, 2006-08-11 04:58 Subject: Re: [PATCH] http-push: Make WebDAV work with (broken?) default apache2 WebDAV module Message-ID: <20060811045801.GA4544@reactrix.com> URL: https://gitlist.dev/e/20060811045801.GA4544%40reactrix.com In-Reply-To: <7vk65h3drd.fsf@assigned-by-dhcp.cox.net> ``` On Wed, Aug 09, 2006 at 12:33:42PM -0700, Junio C Hamano wrote: > > WebDAV on Debian unstable cannot handle renames on WebDAV from > > file.ext to newfile (without ext) when newfile* already > > exists. Normally, git creates a file like 'objects/xx/sha1.token', > > which is renamed to 'objects/xx/sha1' when transferred completely. > > > > Just use '_' instead of '.' so WebDAV doesn't see it as an extension > > change. > > > > Signed-off-by: Rutger Nijlunsing > > Acked-by: Johannes Schindelin > > Good problem and workaround description. Nick, does this look > sane to you, or is there a more kosher way to handle this? This looks good to me - while it seems like the problem is in the DAV server it also seems like problematic DAV servers will be less likely to trip up on an underscore in the filename. -- For a successful technology, reality must take precedence over public relations, for nature cannot be fooled. This message (including any attachments) contains confidential information intended for a specific individual and purpose, and is protected by law. If you are not the intended recipient, you should delete this message. Any disclosure, copying, or distribution of this message, or the taking of any action based on it, is strictly prohibited. ```