Local fs destination is ignored in group downloads

Bug #1224641 reported by Barry Warsaw
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-download-manager
Fix Released
High
Manuel de la Peña
ubuntu-download-manager (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

The wiki documentation says:

createDownloadGroup(IN Arra<GroupDownload> group, IN String algorithm, IN Bool allowGSM, IN Dict<String, Variant> metadata, IN Dict<String, String> headers)

This method allows the creation of a GroupDownload object that will allow to perform the download of a collection of urls so that they are performed as a single atomic operation.
Arguments
IN Array<GroupDownload> group: A group is a structure with the following signature (sss), where the values are:
The url to be downloaded. If this value is empty then the hash will not be checked.
The local file where the download will be stored, please see Download manager special use cases.
The hash of the download.

But with a "local file" destination in /tmp, the files are not moved there after successful download. I get a 'finished' signal with a local_path pointing into ~/.local/share/download_manager/{...} instead. Maybe this is related to the download service only being available on the session bus? LP: #1211486

Related branches

Revision history for this message
Barry Warsaw (barry) wrote :

The problem with this is that there's no way I can accurately move the files from the actual destination to the intended destination. The best I can do - and this is in practice *highly* fragile - is match the last path component of the actual destination against the last path component of the requested url.

Changed in ubuntu-download-manager:
importance: Undecided → High
assignee: nobody → Manuel de la Peña (mandel)
status: New → In Progress
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:ubuntu-download-manager at revision 124, scheduled for release in ubuntu-download-manager, milestone 0.3

Changed in ubuntu-download-manager:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-download-manager - 0.2+13.10.20130920.1-0ubuntu1

---------------
ubuntu-download-manager (0.2+13.10.20130920.1-0ubuntu1) saucy; urgency=low

  [ Barry Warsaw ]
  * When extracting the QPair from _downloadProgress, use a reference
    local variable so that a new QPair isn't created, thus throwing away
    the update. (LP: #1226243)

  [ Manuel de la Pena ]
  * Fixes the irl example in the test folder. (LP: #1221598)
  * Added a class that provides a way to generate secure paths to
    appamor confined applications.
  * Make use of the apparmor class to build secure dbus paths. (LP:
    #1222901)
  * Increases the test coverage of the download factory class.
  * Added the new metadata key 'objectpath' where the client can propose
    a uuid for the download that will be used to create the path. (LP:
    #1215904)
  * Change the interface name so that is less confusing. (LP: #1224538)
  * Fixed the location where the file is stored. If the local-file
    metadata is provided and the app is not cofined that would be the
    used file else the XDG/APP_ID one will be used. (LP: #1224641,
    #1222914)
  * Provide two command line args to be used in tests. (LP: #1202809)
  * Set the accept-encoding header to 'identity' so that we do not
    expand any download. (LP: #1224678)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 126
 -- Ubuntu daily release <email address hidden> Fri, 20 Sep 2013 10:03:28 +0000

Changed in ubuntu-download-manager (Ubuntu):
status: New → Fix Released
Changed in ubuntu-download-manager:
status: Fix Committed → Fix Released
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.