Opened 11 years ago

Last modified 11 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.



Subtickets

Change History (4)

comment:1 Changed 11 years ago by joar

Status: NewIn Progress

comment:1 Changed 11 years ago by joar

Owner: changed from Joar Wandborg to Christopher Webber
Status: In ProgressFeedback
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 Changed 11 years ago by Christopher Allan Webber

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

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



comment:3 Changed 11 years ago by Will Kahn-Greene

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

Note: See TracTickets for help on using tickets.