In case I am right, then this should be a new value in url-encode-path on the store, to ensure compatibility with the 2.0 file store.
/jacob
----- Original Message ----- From: "Oliver Zeigermann" <[EMAIL PROTECTED]>
To: "Slide Users Mailing List" <[EMAIL PROTECTED]>; "Andrew Miller" <[EMAIL PROTECTED]>
Sent: Thursday, November 25, 2004 11:32 AM
Subject: Re: Real filenames in slide tx store
Thanks for the patch, I really like it and would like to see it in 2.1 final. The idea to limit path size to 128 is a nice idea, but you know there is the risk of two different paths to be encooded to the same MD5 hash. Although this is rather unlikely...
As the file system has moved to
http://jakarta.apache.org/commons/transaction/
which has its own release cycle your patch won't make it to Slide 2.1 rc1, but only to the final version.
Oliver
On Wed, 24 Nov 2004 12:02:22 -0500, Andrew Miller <[EMAIL PROTECTED]> wrote:A few months ago I added a new paramater that indicates that portable paths should be used. We were having problems storing Japanese file names on Windows and then migrating the store to Linux.
It uses a technique similar to url-encode-path (base64 encode, then url encode), but it only encodes non-latin paths and encodes each path segment to avoid all files being contained in a single directory. It also limits file length for files stored to the filesytem to 128 characters, using an MD5 hash of the filename to ensure uniqueness when truncating.
I have always meant to contribute this code and it sounds like now may be the time. I have my changes against the SLIDE_2_1_M1_RELEASE tag. I attached the patches for the 2 modified files. I'm happy to create a bug report and if that is preferred.
--Andy
On Wed, 24 Nov 2004 09:52:44 +0100, Jacob Lund <[EMAIL PROTECTED]> wrote:
> The url-encode-path was originally added to enable you to save files > using
> characters not available in your operating system. I did some testing > with
> russian filenames and it only worked when this parameter was set to > true.
>
> /jacob
>
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]