Code

use a hash of the lock token as the suffix for PUT/MOVE
authorTay Ray Chuan <rctay89@gmail.com>
Sat, 14 Feb 2009 09:52:14 +0000 (17:52 +0800)
committerJunio C Hamano <gitster@pobox.com>
Sun, 15 Feb 2009 08:57:43 +0000 (00:57 -0800)
commitdfab7c144eb2f02fc1481173422ecc416a060155
tree5969a39c8fb87bee085cc97c9345b539d04e5dcf
parent2d20b7ebf672c28b18da4558d96f100613e4e4ca
use a hash of the lock token as the suffix for PUT/MOVE

After 753bc91 ("Remove the requirement opaquelocktoken uri scheme"),
lock tokens are in the URI forms in which they are received from the
server, eg. 'opaquelocktoken:', 'urn:uuid:'.

However, "start_put" (and consequently "start_move"), which attempts to
create a unique temporary file using the UUID of the lock token,
inadvertently uses the lock token in its URI form. These file
operations on the server may not be successful (specifically, in
Windows), due to the colon ':' character from the URI form of the lock
token in the file path.

This patch uses a hash of the lock token instead, guaranteeing only
"safe" characters (a-f, 0-9) are used in the file path.

The token's hash is generated when the lock token is received from the
server in handle_new_lock_ctx, minimizing the number of times of
hashing.

Signed-off-by: Tay Ray Chuan <rctay89@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
http-push.c
t/t5540-http-push.sh