http://localhost/ticket/1210 <p> <a class="missing wiki">WebOb?</a> from v1.0.1 has some interesting new behaviour with reading request data for different Content-Types: </p> <ul><li>It seems that looking at request.body mangles request.POST. See fix in ckan cset:e83bad9caa1b </li></ul><ul><li>A particular request caused an <a class="missing wiki">AssertionError?</a> during looking at request.body, so we need to enclose this in a try/except. I couldn't recreate it in a test unfortunately. </li></ul><p> Example: </p> <pre class="wiki">Module ckan.controllers.api:206 in create &lt;&lt; log.debug('create: %s' % (context)) try: request_data = self._get_request_data() except ValueError, inst: response.status_int = 400 &gt;&gt; request_data = self._get_request_data() Module ckan.lib.base:149 in _get_request_data &lt;&lt; cls.log.debug('Retrieving request params: %r' % request.params) cls.log.debug('Retrieving request POST: %r' % request.POST) cls.log.debug('Retrieving request POST body: %r' % request.body) if request.POST: try: &gt;&gt; cls.log.debug('Retrieving request POST body: %r' % request.body) Module paste.registry:137 in __getattr__ &lt;&lt; def __getattr__(self, attr): return getattr(self._current_obj(), attr) def __setattr__(self, attr, value): &gt;&gt; return getattr(self._current_obj(), attr) Module webob.request:470 in _body__get &lt;&lt; Return the content of the request body. """ self.make_body_seekable() # we need this to have content_length r = self.body_file.read(self.content_length) self.body_file.seek(0) &gt;&gt; self.make_body_seekable() # we need this to have content_length Module webob.request:697 in make_body_seekable &lt;&lt; self.body_file_raw.seek(0) else: self.copy_body() &gt;&gt; self.copy_body() Module webob.request:714 in copy_body &lt;&lt; self.body = self.body_file_raw.read(length) elif self.is_body_readable: self.body = self.body_file_raw.read() self._copy_body_tempfile() else: &gt;&gt; self.body = self.body_file_raw.read() Module webob.request:1190 in read &lt;&lt; def read(self, size=-1): body = self._get_body() if size &lt; 0: v = body[self.position:] &gt;&gt; body = self._get_body() Module webob.request:1207 in _get_body &lt;&lt; self._body = _encode_multipart(self.vars, self.content_type) else: assert 0, ('Bad content type: %r' % self.content_type) return self._body &gt;&gt; assert 0, ('Bad content type: %r' % self.content_type) AssertionError: Bad content type: '; charset=utf-8' </pre> en-us http://assets.okfn.org/p/ckan/img/ckan_logo_shortname.png http://localhost/ticket/1210 Trac 0.12.3 dread Wed, 06 Jul 2011 10:51:39 GMT http://localhost/ticket/1210#comment:1 http://localhost/ticket/1210#comment:1 <ul> <li><strong>summary</strong> changed from <em>WebOb 1.0.1 behaviour</em> to <em>POST application/json error handling with newer WebOb</em> </li> </ul> Ticket dread Wed, 06 Jul 2011 10:53:35 GMT http://localhost/ticket/1210#comment:2 http://localhost/ticket/1210#comment:2 <ul> <li><strong>priority</strong> changed from <em>awaiting triage</em> to <em>minor</em> </li> <li><strong>description</strong> modified (<a href="/ticket/1210?action=diff&amp;version=2">diff</a>) </li> </ul> Ticket dread Wed, 06 Jul 2011 17:53:01 GMT http://localhost/ticket/1210#comment:3 http://localhost/ticket/1210#comment:3 <ul> <li><strong>status</strong> changed from <em>new</em> to <em>closed</em> </li> <li><strong>resolution</strong> set to <em>fixed</em> </li> </ul> <p> Fix in ckan for <a class="missing wiki">AssertionError?</a> in cset:8f6ba8ef63f3 lined up for CKAN release 1.4.2. </p> Ticket