apport should ensure description is less than 50k characters

Bug #957326 reported by Brian Murray on 2012-03-16
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
apport (Ubuntu)
Medium
Martin Pitt

Bug Description

I was trying to report a bug regarding update-manager and distribution upgrade from lucid to precise and received and oops report from Launchpad when trying to report the bug.

The oops can be found at https://lp-oops.canonical.com/oops.py/?oopsid=784b24ada04f53ab155d80242075ed70.

Apparently there is limit of 50,000 characters on the description.

11:20 < rick_h> bdmurray: so it looks like LP is rejecting it because the description on the bug report is > 50000 characters long

No matter what I put for the bug description was I able to file the bug report. I think because the data apport had uploaded was making the description longer than the limit. I'll attach the apport "crash" report.

Changed in apport (Ubuntu):
assignee: nobody → Martin Pitt (pitti)
Martin Pitt (pitti) on 2012-03-18
Changed in apport (Ubuntu):
status: New → In Progress
Martin Pitt (pitti) wrote :

Indeed, update-manager adds a field which looks like

Install: sound-theme-freedesktop:amd64 (0.7-0ubuntu3, automatic), libdpkg-perl:amd64 (1.16.1.2ubuntu5, automat
ic), libgconf-2-4:amd64 (3.2.5-0ubuntu1, automatic), python-keyring:amd64 (0.7.1-1fakesync1, automatic), libsws
cale2:amd64 (0.8-1ubuntu2, automatic), libpam-winbind:amd64 (3.6.3-1ubuntu1, automatic), libcdio13:amd64 (0.83-
1, automatic), libmodplug1:amd64 (0.8.8.4-1, automatic), libecryptfs0:amd64 (96-0ubuntu2, automatic), libtokyoc
abinet8:amd64 (1.4.37-6.1build1, automatic), [..]

and is a huge line. I'll fix ProblemReport to not show these inline.

Changed in apport (Ubuntu):
importance: Undecided → Medium
Martin Pitt (pitti) wrote :

Fixed in trunk r2238.

Changed in apport (Ubuntu):
status: In Progress → Fix Committed
Launchpad Janitor (janitor) wrote :
Download full text (3.2 KiB)

This bug was fixed in the package apport - 1.95-0ubuntu1

---------------
apport (1.95-0ubuntu1) precise; urgency=low

  [ Martin Pitt ]
  * New upstream release:
    - apport-gtk, apport-kde: When reporting a "system crash", don't say "...
      of this program version", but "...of this type", as we don't show a
      program version in the initial dialog
      (https://wiki.ubuntu.com/ErrorTracker#error) (LP: #961065)
    - problem_report.py, write_mime(): Do not put a key inline if it is bigger
      than 1 kB, to guard against very long lines. (LP: #957326)
    - etc/cron.daily/apport: Do not remove whoopsie's *.upload* stamps every
      day, only if they are older than a week. whoopsie comes with its own
      cron job which deals with them. Thanks Steve Langasek. (LP: #957102)
    - report.py, mark_ignore(): Fix crash if executable went away underneath
      us. (LP: #961410)
    - apport-gtk: Do not compare current continue button label against a
      translated string. Instead just remember whether or not we can restart
      the application. (LP: #960439)
    - hookutils.py, command_output(): Add option to keep the locale instead of
      disabling it.
    - hookutils.py, command_output(): Actually make the "input" parameter
      work, instead of causing an eternal hang. Add tests for all possible
      modes of operation.
    - hooktuils.py: Change root_command_output() and
      attach_root_command_outputs() to disable translated messages
      (LC_MESSAGES=C) only as part of the command to be run, not already for
      the root prefix command. This will keep the latter (gksu, kdesudo, etc.)
      translated. (LP: #961659)
    - apport-gtk: Cut off text values after 4000 characters, as Gtk's TreeView
      does not get along well with huge values. KDE's copes fine, so continue
      to display the complete value there. (LP: #957062)
    - apport-gtk: Make details window resizable in bug reporting mode.
    - crashdb.py, known(): Check the address signature duplicate database if
      the symbolic signature exists, but did not find any result. (LP: #103083)
    - ui.py: Run anonymization after checking for duplicates, to prevent host
      or user names which look like hex numbers to corrupt the stack trace.
      (LP: #953104)
    - apport-gtk: Require an application to both have TERM and SHELL in its
      environment to consider it a command line application that was started
      by the user. (LP: #962130)
    - backends/packaging-apt-dpkg.py, _check_files_md5(): Fix double encoding,
      which caused UnicodeDecodeErrors on non-ASCII characters in an md5sum
      file. (LP: #953682)
    - apport-kde, apport-gtk: Only show "Relaunch" if the report has a
      ProcCmdline, otherwise we cannot restart it. (LP: #956173)
    - apport-gtk, apport-kde: Show the ExecutablePath while we're collecting
      data for the crash report. Thanks Evan Dandrea. (LP: #938707).
  * debian/copyright: Change to copyright format 1.0.
  * debian/control: Bump Standards-Version to 3.9.3.

  [ Brian Murray ]
  * data/general-hooks/ubuntu.py: use main.log to determine UpgradeStatus not
    apt.log (LP: #886111)
 -- Martin Pitt <<email address hidden>...

Read more...

Changed in apport (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers