Re: [xml] c14n 1.1 support (patch)

2009-08-20 Thread Stefan Behnel
Hi, Aleksey Sanin wrote: Please find attached a patch that adds support for the new version of c14n (http://www.w3.org/TR/xml-c14n11/). I am getting questions about it in the xmlsec mailing list and I finally decided to implement it. I would greatly appreciate if you can accept this patch

Re: [xml] c14n 1.1 support (patch)

2009-07-09 Thread Daniel Veillard
On Thu, Jun 25, 2009 at 01:10:45PM -0700, Aleksey Sanin wrote: Daniel, Hi Aleksey, thanks for the patch, sorry for the delay, I was really focused on work stuff :-\ Please find attached a patch that adds support for the new version of c14n (http://www.w3.org/TR/xml-c14n11/). I am getting

Re: [xml] c14n 1.1 support (patch)

2009-07-09 Thread Aleksey Sanin
thanks for the patch, sorry for the delay, I was really focused on work stuff :-\ No worries, I know how it works :) I don't know if this is nasty or not, but git decided to remove some white spaces, which I'm afraid may change the output of the test, so please double check... I also got a

Re: [xml] c14n 1.1 support (patch)

2009-07-09 Thread Aleksey Sanin
I commited the result, I guess it should be very close to what you had in your tree, but like you I'm learning git so please double check the code in git is actually what you expected :-) (module the small API change). Thanks again, I'll take a look later today and let you know. There were

Re: [xml] c14n 1.1 support (patch)

2009-07-09 Thread Aleksey Sanin
Thanks for committing it! Aleksey Daniel Veillard wrote: On Thu, Jul 09, 2009 at 08:42:12AM -0700, Aleksey Sanin wrote: I commited the result, I guess it should be very close to what you had in your tree, but like you I'm learning git so please double check the code in git is actually what

Re: [xml] c14n 1.1 support (patch)

2009-06-25 Thread Aleksey Sanin
Please ignore the previous patch, it is incomplete. This is the correct one. Sorry, still learning git :) Aleksey Aleksey Sanin wrote: Daniel, Please find attached a patch that adds support for the new version of c14n (http://www.w3.org/TR/xml-c14n11/). I am getting questions about it in the

Re: [xml] c14n 1.1 support (patch)

2009-06-25 Thread Aleksey Sanin
And one more patch that adds extra test vectors and speeds up code a little bit. Aleksey Aleksey Sanin wrote: Please ignore the previous patch, it is incomplete. This is the correct one. Sorry, still learning git :) Aleksey Aleksey Sanin wrote: Daniel, Please find attached a patch that