Fix function package downloading

Use function id as the default function package name for downloading.

Change-Id: I4b7b6e837ca74dae079b11ace235e635990545d0
Story: 2002015
Task: 19655

qinling/api/controllers/v1/function.py

@@ -14,7 +14,6 @@
 
 import collections
 import json
-import os
 
 from oslo_config import cfg
 from oslo_log import log as logging
@@ -107,23 +106,24 @@ class FunctionsController(rest.RestController):
         This method can support HTTP request using either
         'Accept:application/json' or no 'Accept' header.
         """
-        LOG.info("Get function %s.", id)
+        ctx = context.get_ctx()
+        acl.enforce('function:get', ctx)
 
         download = strutils.bool_from_string(
             pecan.request.GET.get('download', False)
         )
         func_db = db_api.get_function(id)
-        ctx = context.get_ctx()
 
         if not download:
+            LOG.info("Getting function %s.", id)
             pecan.override_template('json')
             return resources.Function.from_db_obj(func_db).to_dict()
-        else:
+
         LOG.info("Downloading function %s", id)
         source = func_db.code['source']
 
         if source == constants.PACKAGE_FUNCTION:
-                f = self.storage_provider.retrieve(ctx.projectid, id,
+            f = self.storage_provider.retrieve(func_db.project_id, id,
                                                func_db.code['md5sum'])
         elif source == constants.SWIFT_FUNCTION:
             container = func_db.code['swift']['container']
@@ -141,7 +141,7 @@ class FunctionsController(rest.RestController):
                                    else FileIter(f))
         pecan.response.headers['Content-Type'] = 'application/zip'
         pecan.response.headers['Content-Disposition'] = (
-                'attachment; filename="%s"' % os.path.basename(func_db.name)
+            'attachment; filename="%s"' % id
         )
 
     @rest_utils.wrap_pecan_controller_exception

qinling/storage/file_system.py

@@ -89,7 +89,7 @@ class FileSystemStorage(base.PackageStorage):
         LOG.debug(
             'Getting package data, function: %s, version: %s, md5sum: %s, '
             'project: %s',
-            function, md5sum, version, project_id
+            function, version, md5sum, project_id
         )
 
         if version != 0: