RE: svn + isa (firewall) + https

Hi Hamid,

Good catch. I verified that this string must be checked case insensitive
(RFC2617) and then applied a patch in r35981.

Thanks,

   Bert

From: Hamid Jahanzad [mailto:HamidJ_at_microsol.ie]
Sent: donderdag 19 februari 2009 13:52
To: dev_at_subversion.tigris.org
Subject: svn + isa (firewall) + https

Dear dev!

Svn clients (1.5.5) on linux can not connect to svn server behind Microsoft
ISA firewall publishing the svn server using https web listener.

This is due to the fact that ISA server publishes the attribute names using
capital letter, in this case "Realm". However svn server publish them as
all lower case "realm". Subversion/libsvn_ra_serf/auth.c compares the
attributes using "strcmp" which is a case sensitive comparison .

Solution:

By replacing the "strcmp" with "strcasecmp" (case insensitive compare),
problem goes away!

Regards,

Hamid Jahanzad

_______________________________________________________________________

This email and any files transmitted with it are confidential and intended
solely for the use of the individual or entity to which they are addressed.

If you have received this email in error please notify the originator of the
message.
This footer also confirms that this email message has been scanned for the
presence of computer viruses.

Any views expressed in this message are those of the individual sender,
except where the sender specifies and with authority, states them to be the
views of Microsol.

If you have received this mail in error you are requested to e-mail us
(info_at_microsol.ie) or by telephoning Microsol on +353 14153700.

Please Visit our website at
http://www.microsol.com

_______________________________________________________________________

This email has been scanned for all viruses.
_______________________________________________________________________

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1191868
Received on 2009-02-19 14:40:57 CET