Comment 5 for bug 914779

Revision history for this message
Colin Watson (cjwatson) wrote :

Attempting to call newPocketUploader fails:

  Module zope.publisher.publish, line 134, in publish
    result = publication.callObject(request, obj)
  Module lazr.restful.publisher, line 211, in callObject
    WebServicePublicationMixin, self).callObject(request, object)
  Module lp.services.webapp.publication, line 499, in callObject
    return mapply(ob, request.getPositionalArguments(), request)
  Module zope.publisher.publish, line 109, in mapply
    return debug_call(obj, args)
   - __traceback_info__: <security proxied lazr.restful._resource.EntryResource instance at 0x1140daec>
  Module zope.publisher.publish, line 115, in debug_call
    return obj(*args)
  Module lazr.restful._resource, line 925, in __call__
    result = self.do_POST()
  Module lazr.restful._resource, line 757, in do_POST
    return self.handleCustomPOST(operation_name)
  Module lazr.restful._resource, line 1396, in handleCustomPOST
    value = super(EntryResource, self).handleCustomPOST(operation_name)
  Module lazr.restful._resource, line 741, in handleCustomPOST
    return operation()
  Module lazr.restful._operation, line 83, in __call__
    response = self.call(**values)
  Module lazr.restful.declarations, line 1429, in call
    response.setHeader('Location', absoluteURL(result, self.request))
  Module zope.traversing.browser.absoluteurl, line 35, in absoluteURL
    return zope.component.getMultiAdapter((ob, request), IAbsoluteURL)()
  Module lp.services.webapp.publisher, line 617, in __str__
    return canonical_url(self.context, self.request)
  Module lp.services.webapp.publisher, line 681, in canonical_url
    if urldata.path]
  Module lp.soyuz.browser.archivepermission, line 53, in path
    "One of component, sourcepackagename or package set should "
AssertionError: One of component, sourcepackagename or package set should be set

I'll fix this, which looks easy enough, but it's clearly a gap in my testing. That said, I don't think any existing functionality is broken, so this probably doesn't prevent this patch being rolled out.