"dump_dir" should be optional when creating serializers if "fh" is given

Bug #471755 reported by Natalia Bidart
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
wsgi-oops
Confirmed
Low
Ubuntu One Foundations+ team

Bug Description

Current initialization for OOPSFileSerializer is:

    def __init__(self, key, dump_dir, fh=None):
        super(OOPSFileSerializer, self).__init__(key)
        self._dump_dir = dump_dir
        self._filename = None
        self._oops = None
        self._fh = fh
        if not os.path.isdir(dump_dir):
            os.makedirs(dump_dir)

if fh is defined, the serializer shouldn't require a mandatory "dump_dir" and it shouldn't create the dir either.

Changed in wsgi-oops:
assignee: nobody → Naty Bidart (nataliabidart)
importance: Undecided → Low
status: New → Confirmed
Changed in wsgi-oops:
assignee: Naty Bidart (nataliabidart) → Ubuntu One Foundations+ team (ubuntuone-foundations+)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.