From 10caa71b97f06f7ea51769c59ccd1309d81c5f14 Mon Sep 17 00:00:00 2001
From: Michael Hanke <michael.hanke@gmail.com>
Date: Thu, 30 Sep 2021 17:09:07 +0200
Subject: [PATCH] Inherit URL and credentials name config in subdatasets

This is criticial for authed-access.

Notes are inside how this could be done better. But for now, speed is
more important.
---
 datalad_xnat/update.py | 37 +++++++++++++++++++++++++++----------
 1 file changed, 27 insertions(+), 10 deletions(-)

diff --git a/datalad_xnat/update.py b/datalad_xnat/update.py
index 9209abc..a5b6954 100644
--- a/datalad_xnat/update.py
+++ b/datalad_xnat/update.py
@@ -97,6 +97,7 @@ def __call__(project=None,
                  jobs='auto',
                  dataset=None):
 
+
         ds = require_dataset(
             dataset, check_installed=True, purpose='update')
 
@@ -161,6 +162,7 @@ def __call__(project=None,
             # go with a single big one
             subjects = [None]
         from datalad_xnat.parser import parse_xnat
+        from unittest.mock import patch
         for sub in subjects:
             try:
                 # all this tempfile madness is only needed because windows
@@ -189,16 +191,31 @@ def __call__(project=None,
                 # corresponds to the header field 'filename' in the csv table
                 filename = '{filename}'
                 filenameformat = f"{pathfmt}{filename}"
-                ds.addurls(
-                    str(addurls_table_fname), '{url}', filenameformat,
-                    ifexists=ifexists,
-                    fast=True if reckless == 'fast'
-                    else False,
-                    save=True,
-                    jobs=jobs,
-                    cfg_proc=None if platform.credential_name == 'anonymous'
-                    else 'xnat_dataset',
-                    result_renderer='default')
+                # shoehorn essential info into the ENV to make it
+                # accessible to the config procedure
+                # TODO maybe alter the config procedure to pull this info
+                # from a superdataset, if it finds the dataset at hand
+                # unconfigured.
+                env_prefix = f'DATALAD_XNAT_{xnat_cfg_name.upper()}'
+                env = {
+                    'DATALAD_XNAT_DEFAULT__NAME': xnat_cfg_name,
+                    f'{env_prefix}_URL': platform.url,
+                }
+                if platform.credential_name != 'anonymous':
+                    env[f'{env_prefix}_CREDENTIAL__NAME'] = \
+                        platform.credential_name
+                with patch.dict('os.environ', env):
+                    ds.addurls(
+                        str(addurls_table_fname), '{url}', filenameformat,
+                        ifexists=ifexists,
+                        fast=True if reckless == 'fast'
+                        else False,
+                        save=True,
+                        jobs=jobs,
+                        cfg_proc=None
+                        if platform.credential_name == 'anonymous'
+                        else 'xnat_dataset',
+                        result_renderer='default')
             finally:
                 if addurls_table_fname.exists():
                     addurls_table_fname.unlink()