Bug #29187 for Business-PayPal-EWP: s/,/\n/g not flexible enough

Wed Sep 05 10:23:32 2007 GHENRY [...] cpan.org - Ticket created

Subject:

s/,/\n/g not flexible enough

Please consider something like Text::CSV_XS. Also, the resulting encrypted data should be all on one line, as per Paypal examples: -----BEGIN PKCS7-----MIIGyg......ILiOUhJpCo=-----END PKCS7-----"/></form> There are many newlines, and I'm having to run $encrypted =~ s/\n//g; on it. Thanks.

Wed Sep 05 11:04:53 2007 isaac [...] cpan.org - Correspondence added

Hi there, The encrypted output should actually not be on one line. PayPal is nice and will accept it, but the docs say: "PEM requires that all encoded lines consist of exactly 64 printable characters, with the exception of the last line, which may contain fewer printable characters. Lines are delimited by whitespace characters according to local (platform-specific) conventions." Regarding the regexp, it's not stricly necessary but as "," characters can't be part of an application/x-www-form-urlencoded payload (they'd need to be encoded) anyway, it was done that way for convenience. I don't plan on changing the format (as it will break old software) nor do I want to add dependencies. I'd want stonger evidence before I changed it. Issac On Wed Sep 05 10:23:32 2007, GHENRY wrote: Show quoted text

> Please consider something like Text::CSV_XS. > > Also, the resulting encrypted data should be all on one line, as per > Paypal examples: > > -----BEGIN PKCS7-----MIIGyg......ILiOUhJpCo=-----END PKCS7-----

"/></form> Show quoted text

> > There are many newlines, and I'm having to run $encrypted =~ s/\n//g;

on it. Show quoted text

> > Thanks.

--

Wed Sep 05 11:04:55 2007 The RT System itself - Status changed from 'new' to 'open'

Wed Sep 05 11:05:36 2007 isaac [...] cpan.org - Status changed from 'open' to 'rejected'

Wed Sep 05 11:05:41 2007 isaac [...] cpan.org - Given to ISAAC

Wed Sep 05 11:16:30 2007 GHENRY [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #29187] s/,/\n/g not flexible enough
Date:	Wed, 5 Sep 2007 16:11:17 +0100 (BST)
To:	bug-Business-PayPal-EWP [...] rt.cpan.org
From:	"Gavin Henry" <ghenry [...] cpan.org>

<quote who="via RT"> Show quoted text

> > <URL: http://rt.cpan.org/Ticket/Display.html?id=29187 > > > Hi there, > The encrypted output should actually not be on one line. PayPal is > nice and will accept it, but the docs say: > > "PEM requires that all encoded lines consist of exactly 64 printable > characters, with the exception of the last line, which may contain > fewer printable characters. Lines are delimited by whitespace > characters according to local (platform-specific) conventions."

Ah, I must have missed that. Thanks for taking the time to find it. Show quoted text

> > Regarding the regexp, it's not stricly necessary but as "," characters > can't be part of an application/x-www-form-urlencoded payload (they'd > need to be encoded) anyway, it was done that way for convenience. I > don't plan on changing the format (as it will break old software) nor > do I want to add dependencies. I'd want stonger evidence before I > changed it.

I'll take the , out of my item_name then ;-) Thanks for your time and replies. Gavin. Show quoted text

> > Issac > > On Wed Sep 05 10:23:32 2007, GHENRY wrote:

>> Please consider something like Text::CSV_XS. >> >> Also, the resulting encrypted data should be all on one line, as per >> Paypal examples: >> >> -----BEGIN PKCS7-----MIIGyg......ILiOUhJpCo=-----END PKCS7-----

> "/></form>

>> >> There are many newlines, and I'm having to run $encrypted =~ s/\n//g;

> on it.

>> >> Thanks.

> > -- > >

Wed Sep 05 11:16:32 2007 The RT System itself - Status changed from 'rejected' to 'open'

Wed Sep 05 11:20:59 2007 isaac [...] cpan.org - Correspondence added

I'd actually be curious to know if wrapping the problematic name with uri_escape() [use URI::Escape] doesn't make things better for you? Issac

Wed Sep 05 12:53:39 2007 GHENRY [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #29187] s/,/\n/g not flexible enough
Date:	Wed, 5 Sep 2007 17:48:47 +0100 (BST)
To:	bug-Business-PayPal-EWP [...] rt.cpan.org
From:	"Gavin Henry" <ghenry [...] cpan.org>

<quote who="via RT"> Show quoted text

> > <URL: http://rt.cpan.org/Ticket/Display.html?id=29187 > > > I'd actually be curious to know if wrapping the problematic name with > uri_escape() [use URI::Escape] doesn't make things better for you? > > Issac >

Thanks. I've just got to get my buttons working first ;-) Not liking some params. More tomorrow.

Fri Feb 04 06:44:24 2011 isaac [...] cpan.org - Status changed from 'open' to 'stalled'