Roundup Tracker - Issues


Author rouilj
Recipients rouilj
Date 2019-06-07.01:38:43
Message-id <>
Hi Joseph,

Sorry to bother you but I bisected the code and it looks like the 
commit rev5671:f60c44563c3a makes image file uploads and large (967 
works, 1303 fails) messages fail when running under python2:

changeset:   5671:f60c44563c3a
user:        Joseph Myers <>
date:        Sun Mar 24 21:49:17 2019 +0000
files:       roundup/cgi/
Adjust make_file override to use binary files only when needed.

Previous version would produce errors for large text fields in form
submissions with Python 3, so inherit from the default make_file
implementation and only force binary files in the specific case
(self.length >= 0) identified in <>.

So it looks like this error is now happening under python2 post commit.

Running python2 nuke and attaching a file or adding a largish 
message while creating a new issue (or adding to an existing issue) 
causes the following crash:

EXCEPTION AT Fri Jun  7 01:26:07 2019
Traceback (most recent call last):
  File "roundup.hg/roundup/scripts/", line 222, in 
  File "roundup.hg/roundup/scripts/", line 459, in 
    tracker.Client(tracker, self, env).main()
  File "roundup.hg/roundup/cgi/", line 395, in __init__
    self.form = BinaryFieldStorage(fp=request.rfile, environ=env)
  File "/usr/lib/python2.7/", line 507, in __init__
    self.read_multi(environ, keep_blank_values, strict_parsing)
  File "/usr/lib/python2.7/", line 636, in read_multi
    environ, keep_blank_values, strict_parsing)
  File "/usr/lib/python2.7/", line 509, in __init__
  File "/usr/lib/python2.7/", line 646, in read_single
  File "/usr/lib/python2.7/", line 668, in read_lines
  File "/usr/lib/python2.7/", line 728, in 
    self.__write(odelim + line)
  File "/usr/lib/python2.7/", line 675, in __write
    self.file = self.make_file('')
  File "roundup.hg/roundup/cgi/", line 246, in make_file
    return super().make_file()
TypeError: super() takes at least 1 argument (0 given)

Not sure what to do here. Should it be super().make_file(self)?

At the very least it looks like we have no test for this code path.

Date User Action Args
2019-06-07 01:38:44rouiljsetrecipients: + rouilj
2019-06-07 01:38:44rouiljsetmessageid: <>
2019-06-07 01:38:43rouiljlinkissue2551046 messages
2019-06-07 01:38:43rouiljcreate