tahoe-lafs/trac-2024-07-25

upcoming nevow breakage #2732

New issue

Closed

opened 2016-02-16 17:33:46 +00:00 by warner · 3 comments

warner commented

2016-02-16 17:33:46 +00:00

Owner

The current version of Nevow (0.12.0) uses a feature of Twisted (request.received_headers) that was deprecated in Twisted-13.2.0 and will be removed entirely in the upcoming Twisted-16.0.0 .

When it is removed, unit tests (and almost certainly all other functionality) will fail with error messages that look like this:

  File "/Users/warner/stuff/tahoe/tahoe/ve/lib/python2.7/site-packages/twisted/web/http.py", line 1736, in allContentReceived
    req.requestReceived(command, path, version)
  File "/Users/warner/stuff/tahoe/tahoe/src/allmydata/webish.py", line 81, in requestReceived
    self.process()
  File "/Users/warner/stuff/tahoe/tahoe/ve/lib/python2.7/site-packages/nevow/appserver.py", line 146, in process
    self.fields = cgi.FieldStorage(self.content, self.received_headers,
exceptions.AttributeError: MyRequest instance has no attribute 'received_headers'

The Nevow maintainers are aware of this nevow#53, and with luck they'll make a new release before Twisted-16.0.0 comes out. If so, we just need to add the new Nevow to our deps tarball and we're good. We might want to increase our dependency on Nevow as well, to force an upgrade.

This ticket is mostly here in case Twisted releases before Nevow, so we'll have an idea of what the problem is.

The current version of Nevow (0.12.0) uses a feature of Twisted (`request.received_headers`) that was deprecated in Twisted-13.2.0 and will be removed entirely in the upcoming Twisted-16.0.0 . When it is removed, unit tests (and almost certainly all other functionality) will fail with error messages that look like this: ``` File "/Users/warner/stuff/tahoe/tahoe/ve/lib/python2.7/site-packages/twisted/web/http.py", line 1736, in allContentReceived req.requestReceived(command, path, version) File "/Users/warner/stuff/tahoe/tahoe/src/allmydata/webish.py", line 81, in requestReceived self.process() File "/Users/warner/stuff/tahoe/tahoe/ve/lib/python2.7/site-packages/nevow/appserver.py", line 146, in process self.fields = cgi.FieldStorage(self.content, self.received_headers, exceptions.AttributeError: MyRequest instance has no attribute 'received_headers' ``` The Nevow maintainers are aware of this [nevow#53](https://github.com/twisted/nevow/issues/53#issuecomment-184691874), and with luck they'll make a new release before Twisted-16.0.0 comes out. If so, we just need to add the new Nevow to our deps tarball and we're good. We might want to increase our dependency on Nevow as well, to force an upgrade. This ticket is mostly here in case Twisted releases before Nevow, so we'll have an idea of what the problem is.

tahoe-lafs added the

labels 2016-02-16 17:33:46 +00:00

mithrandi commented

2016-02-17 15:16:33 +00:00

Author

Owner

Nevow 0.13.0 is now out. I'd appreciate it if someone could check whether we broke anything for Tahoe-LAFS, but hopefully everything should be fine.

warner commented

2016-02-17 17:24:51 +00:00

Author

Owner

I just ran tests with the latest version. Works great. And the only remaining deprecation warning is one of our own (#2734, where we use conch in the 'manhole' module).

I've replaced nevow-0.11.1 with 0.13.0 in the tahoe-deps tarball, so the next "SUMO" tarball will include it.

Thanks for the quick release!

I just ran tests with the latest version. Works great. And the only remaining deprecation warning is one of our own (#2734, where we use conch in the 'manhole' module). I've replaced nevow-0.11.1 with 0.13.0 in the tahoe-deps tarball, so the next "SUMO" tarball will include it. Thanks for the quick release!

tahoe-lafs added the

fixed

label 2016-02-17 17:24:51 +00:00

warner closed this issue

2016-02-17 17:24:51 +00:00

warner commented

2016-03-22 05:03:36 +00:00

Author

Owner

Milestone renamed

tahoe-lafs added this to the 1.11.0 milestone 2016-03-22 05:03:36 +00:00