Beefy Boxes and Bandwidth Generously Provided by pair Networks
Pathologically Eclectic Rubbish Lister
 
PerlMonks  

Re: A patched CGI::Simple for REST applications - comments please

by jaldhar (Vicar)
on Aug 22, 2010 at 14:43 UTC ( #856590=note: print w/ replies, xml ) Need Help??


in reply to A patched CGI::Simple for REST applications - comments please

Hello again, I've created a module called REST::Utils which contains a get_body function based in part on our last conversation. It is considerably simpler than the code you have shown above but I wonder if you would like to try it and give it a critique. (And maybe patches if it can be adapted to your needs?)

--
જલધર


Comment on Re: A patched CGI::Simple for REST applications - comments please
Re^2: A patched CGI::Simple for REST applications - comments please
by isync (Hermit) on Aug 23, 2010 at 10:23 UTC
    From a glance at the code, it seems a handy module when doing REST with CGI.pm-like modules. Good that you released it and the insight it provides along with it to the puplic!

    What remains is the handling of large uploads by handle only. Admittedly, I handled it a bit heavy handed in my patching of CGI::Simple. Still, I see no alternative to switching between handling smaller requests by slurping it all into a var, while larger requests need to remain on STDIN and be read() buffered. Can be done by wrapping your get_body() method with an if/else switch based on POST_MAX.
    Or could be that the abstractions that modules like CGI.pm provide are by design opposites of the low-level access my specific scripts require...

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://856590]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others contemplating the Monastery: (8)
As of 2014-07-31 23:05 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    My favorite superfluous repetitious redundant duplicative phrase is:









    Results (255 votes), past polls