[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: start-commit bug?

From: Karl Fogel <kfogel_at_red-bean.com>
Date: Mon, 21 Jul 2008 15:56:44 -0400

"Jay Flaherty" <jflaherty_at_convio.com> writes:
> I came across what I think is a bug with the start-commit hook. Have not found
> evidence that this is a know bug.
> Submitting here before I post to the dev list.
> I am running ver. 1.5.0 of the server and client.
> I have authentication on for select users with write access
> I have a start-commit hook that checks the capabilities parameter for the
> 'mergeinfo' string.
> This hook script rejects the commit and throws an error if this string does not
> exist.
> Everything works fine if you are already authenticated but if you need to
> authenticate then
> the capabilities parameter is empty and the start-hook an error.

It looks like we made some changes related to this in r31620 and r31622,
but they were just about fixing a bug on the server to avoid a crash (or
something) when the client fails to report capabilities. There was no
fix for the bug of the client not reporting the capabilities in the
first place.

I'm not sure why we're not reporting capabilities. Could you file this
as an issue, so at least we have a place to track it? (I'm busy working
on issue #2843 right now, so can't look into this.) Link back to this
thread, include the reproduction recipe directly in the issue, and put
the start-commit hook on the issue as an attachment.

By the way, I searched and didn't find an existing issue for this. I
don't know if it's a regression or not; if you have time to test with
older releases and find out, that would be a huge help.


To unsubscribe, e-mail: users-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: users-help_at_subversion.tigris.org
Received on 2008-07-21 21:57:14 CEST

This is an archived mail posted to the Subversion Users mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.