Opened 12 years ago

Last modified 12 years ago

#242 closed defect (FIXED)

Closing storage objects

Reported by: Elrond Owned by: joar
Priority: minor Milestone: 0.1.0
Component: programming Keywords:
Cc: Parent Tickets:

Description 

As I don't have any simple way to test cloud storage, I'm filing
this as a bug.


1. StorageObjectWrapper should have a close() method
2. ``process_media/__init__.py`` should use the real ``closing()``
   function from ``contextlib`` instead of the current dummy one.

I would also suggest to rename StorageObjectWrapper to include
something about CloudFiles.

Change History (4)

comment:1 by joar, 12 years ago

Status: New → In Progress

comment:1 by joar, 12 years ago

Owner: changed from Joar Wandborg to Christopher Webber
Status: In Progress → Feedback 
There is now
`a fix <https://github.com/jwandborg/mediagoblin/commit/c2b862d1349b5ecc6d787000edb209525b719751>`_
in my branch
`https://github.com/jwandborg/mediagoblin/tree/f571\_closing\_storage\_objects <https://github.com/jwandborg/mediagoblin/tree/f571_closing_storage_objects>`_

Cheers,

comment:2 by Christopher Allan Webber, 12 years ago

Milestone: → 0.1.0
Owner: changed from Christopher Webber to Joar Wandborg
Status: Feedback → Closed 
Added exit to the cloudstorage meta-file.. nice thinking!

Thanks Elrond, Joar... merged, closed, pushed!

comment:3 by Will Kahn-Greene, 12 years ago

The original url for this bug was ​http://bugs.foocorp.net/issues/571 .

Note: See TracTickets for help on using tickets.