[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [virt-tools-list] [PATCH virt-viewer] file: add delete-this-file option

On Apr 30, 2013, at 11:34 , Christophe Fergeau <cfergeau redhat com> wrote:

> On Tue, Apr 30, 2013 at 02:33:12AM +0200, Marc-André Lureau wrote:
>> In case the virt-viewer setting file is meant to temporary, you may use
>> the delete-this-file=1 option to ask the client to remove it, once it
>> has been read. This is useful for example in ovirt context, where
>> connection settings file are generated and can't be reused.
> Not sure what to think about that one, as in the ovirt case, the file will
> be downloaded from user portal, and will also appear in the list of files
> downloaded by the browser (I think).
It will appear there

> So this patch just looks like it's
> papering over a more fundamental issue, and not in a very elegant way imo.
However the biggest issue is creating the files in (in chrome's case) in user's Downloads directory. So this patch would actually help a lot in keeping that clean
There is no possible way how to remove a file from within the browser/portal and only a standalone invoked application can do that. For that purpose remote-viewer is an ideal solution
Unfortunately browser do not have a way how to just "open" a file, it always needs to create a temp file on the user's side. And in case of Chrome they don't even make it temporary.

> Christophe

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]