Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

webob ignores parameters for put/delete #18

Closed
GoogleCodeExporter opened this issue Mar 24, 2015 · 3 comments
Closed

webob ignores parameters for put/delete #18

GoogleCodeExporter opened this issue Mar 24, 2015 · 3 comments

Comments

@GoogleCodeExporter
Copy link

The current version of WebOb found in App Engine ignores form parameters for 
non GET/POST requests, which is really inconvenient for RESTfull apps.

I hope that a newer version does ship in the same release as 2.7 (that also 
includes this framework AFAIK), but for the time being, I was tired on doing 
workarounds so I made a really really simple override in webapp2 to handle this 
issue:

@property
def str_POST(self):
    method = self.method
    self.method = 'POST'
    vars = super(Request, self).str_POST
    self.method = method
    return vars

Very straightforward. Yes, it's a hack, but I think it gives the behavior 
people expect.

Anyway, thought you might be interested. And of course, thank you for all the 
amazing job you've done :)

Original issue reported on code.google.com by jaime.go...@gmail.com on 5 Sep 2011 at 5:21

@GoogleCodeExporter
Copy link
Author

In webOb lib version 0.9.3 PUT params are supported in request.POST dict with 
request Content Type 'application/x-www-form-urlencoded' or 
'multipart/form-data'.

Try to replace webOb lib version in SDK 1.6.0: today is 0.9. AppEngine 
documentation says thah webOb lib version supported is 1.1.1. I think that in 
production it should work.

Original comment by carlos.e...@gmail.com on 15 Nov 2011 at 11:33

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Yes, this is no longer an issue with the new SDK :D

Original comment by jaime.go...@gmail.com on 16 Nov 2011 at 12:19

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

I'm going to mark this as fixed since it works with the Python 2.7 runtime.

Original comment by bquin...@google.com on 5 Dec 2011 at 11:28

  • Changed state: Fixed
  • Added labels: ****
  • Removed labels: ****

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant