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

Michal Skrivanek mskrivan at redhat.com
Tue Apr 30 09:36:57 UTC 2013


On Apr 30, 2013, at 11:34 , Christophe Fergeau <cfergeau at 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.

Thanks,
michal
> 
> Christophe





More information about the virt-tools-list mailing list