add action attr to providers

 * Inform the provider what action is going to be performed,
   currently either 'render' or 'export'.  The action is not used by
   default, but future commits to the osf provider will make use of
   it.

Author: felliott

mfr/core/provider.py

@@ -14,7 +14,7 @@ class BaseProvider(metaclass=abc.ABCMeta):
     `mfr.server.settings.ALLOWED_PROVIDER_DOMAINS`.
     """
 
-    def __init__(self, request, url):
+    def __init__(self, request, url, action=None):
         self.request = request
         url_netloc = furl.furl(url).netloc
         if url_netloc not in settings.ALLOWED_PROVIDER_NETLOCS:
@@ -25,6 +25,7 @@ def __init__(self, request, url):
                 code=400
             )
         self.url = url
+        self.action = action
         self.provider_metrics = MetricsRecord('provider')
         self.metrics = self.provider_metrics.new_subrecord(self.NAME)
 

mfr/core/utils.py

@@ -4,7 +4,7 @@
 from mfr.core import exceptions
 
 
-def make_provider(name, request, url):
+def make_provider(name, request, url, action=None):
     """Returns an instance of :class:`mfr.core.provider.BaseProvider`
 
     :param str name: The name of the provider to instantiate. (osf)
@@ -19,6 +19,7 @@ def make_provider(name, request, url):
             name=name.lower(),
             invoke_on_load=True,
             invoke_args=(request, url, ),
+            invoke_kwds={'action': action},
         ).driver
     except RuntimeError:
         raise exceptions.MakeProviderError(

mfr/providers/osf/provider.py

@@ -30,8 +30,8 @@ class OsfProvider(provider.BaseProvider):
     UNNEEDED_URL_PARAMS = ('_', 'token', 'action', 'mode', 'displayName')
     NAME = 'osf'
 
-    def __init__(self, request, url):
-        super().__init__(request, url)
+    def __init__(self, request, url, action=None):
+        super().__init__(request, url, action)
         self.download_url = None
         self.headers = {}
 

mfr/server/handlers/core.py

@@ -114,7 +114,8 @@ async def prepare(self):
         self.provider = utils.make_provider(
             settings.PROVIDER_NAME,
             self.request,
-            self.url
+            self.url,
+            action=self.NAME,
         )
 
         self.metadata = await self.provider.metadata()