Skip Menu |

This queue is for tickets about the Net-Amazon-EC2 CPAN distribution.

Report information
The Basics
Id: 43235
Status: resolved
Priority: 0/
Queue: Net-Amazon-EC2
People
Owner: Nobody in particular
Requestors: aar [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 0.09
Fixed in: 0.11

History Show all quoted text
  Thu Feb 12 09:37:01 2009 aar [...] cpan.org - Ticket created
Subject: Usage of GET instead of POST limits UserData to 8kb
The parameter UserData in run_instances may contain up to 16kb according to the EC2 spec. However, if we send the request through GET, we run into a 8kb limit for the query string. Thus getting a fatal server error on run_instances. I easily fixed this by switching to HTTP POST. See around line 156 of EC2.pm: 1) remove the line: $uri->query_form(\%params); - replace the line: my $res = $ua->get($ur); with: my $res = $ua->post($ur, \%params); That's all.
  Sat May 09 12:03:56 2009 dohyun [...] hollow.org - Status changed from 'new' to 'open'
  Thu Sep 24 00:55:57 2009 dohyun [...] hollow.org - Status changed from 'open' to 'resolved'
  Thu Sep 24 00:55:58 2009 dohyun [...] hollow.org - Fixed in 0.10 added
  Wed Oct 07 09:50:54 2009 aar [...] cpan.org - Correspondence added
On Gio. 12 Feb. 2009 09:37:01, AAR wrote: Show quoted text
> The parameter UserData in run_instances may contain up to 16kb > according to the EC2 spec. > However, if we send the request through GET, we run into a 8kb limit > for the query string. > Thus getting a fatal server error on run_instances. > > I easily fixed this by switching to HTTP POST. > > See around line 156 of EC2.pm: > > 1) remove the line: > $uri->query_form(\%params); > > - replace the line: > my $res = $ua->get($ur); > with: > my $res = $ua->post($ur, \%params); > > That's all.
Thank you, however the fix is incomplete and broke things. Form params need to passed to the $ua->post() method and no more inserted in the URI (note that forms submitted via POST instead of GET do not pass their values in the query string). I fear that the module is currently broken. See my post above in this ticket for a complete bugfix.
  Wed Oct 07 09:50:55 2009 The RT System itself - Status changed from 'resolved' to 'open'
  Sat Nov 14 00:39:55 2009 dohyun [...] hollow.org - Status changed from 'open' to 'resolved'
  Sat Nov 14 00:39:55 2009 dohyun [...] hollow.org - Fixed in 0.11 added
  Sat Nov 14 00:39:55 2009 dohyun [...] hollow.org - Fixed in 0.10 deleted