From 38073eb2365e5c7a6b8c32265e6cd6456a42080e Mon Sep 17 00:00:00 2001 From: Igor Octaviano Date: Mon, 5 Oct 2026 12:31:36 -0300 Subject: [PATCH] fix: use idc-dicomweb as default data source instead of merge The merge data source was causing "no DICOMweb base url provided" errors because it always tries to initialize ALL child data sources, including the GCP data source which has no URL when there's no ?gcp= query param. Change defaultDataSourceName from 'idc-merge' to 'idc-dicomweb' in: - idc-assets/app-config-template.js (production deployment) - platform/app/public/config/default.js (local dev) - platform/app/public/config/idc.js (local dev) The GCP extension will dynamically activate a merge data source when the ?gcp= query param is present. --- idc-assets/app-config-template.js | 7 +++---- platform/app/public/config/default.js | 7 +++---- platform/app/public/config/idc.js | 7 +++---- 3 files changed, 9 insertions(+), 12 deletions(-) diff --git a/idc-assets/app-config-template.js b/idc-assets/app-config-template.js index ba6b8f6f016..3e2b9ef572e 100644 --- a/idc-assets/app-config-template.js +++ b/idc-assets/app-config-template.js @@ -79,11 +79,10 @@ window.config = { prefetch: 25, }, /** - * Default data source - uses merge to combine IDC + GCP. - * When no ?gcp= param, only IDC data is shown. - * When ?gcp= param is present, both sources are merged. + * Default data source - uses IDC DICOMWeb directly. + * When ?gcp= param is present, the GCP extension activates the merge data source. */ - defaultDataSourceName: 'idc-merge', + defaultDataSourceName: 'idc-dicomweb', dataSources: [ /** * IDC's primary DICOMWeb server (static WADO). diff --git a/platform/app/public/config/default.js b/platform/app/public/config/default.js index e47db26a510..4cc9a4d8c96 100644 --- a/platform/app/public/config/default.js +++ b/platform/app/public/config/default.js @@ -117,11 +117,10 @@ window.config = { showErrorDetails: 'always', /** - * Default data source - uses merge to combine IDC + GCP. - * When no ?gcp= param, only IDC data is shown. - * When ?gcp= param is present, both sources are merged. + * Default data source - uses IDC DICOMWeb directly. + * When ?gcp= param is present, the GCP extension activates the merge data source. */ - defaultDataSourceName: 'idc-merge', + defaultDataSourceName: 'idc-dicomweb', dataSources: [ /** diff --git a/platform/app/public/config/idc.js b/platform/app/public/config/idc.js index 0bceea7c083..3d6758d2eb5 100644 --- a/platform/app/public/config/idc.js +++ b/platform/app/public/config/idc.js @@ -117,11 +117,10 @@ window.config = { showErrorDetails: 'always', /** - * Default data source - uses merge to combine IDC + GCP. - * When no ?gcp= param, only IDC data is shown. - * When ?gcp= param is present, both sources are merged. + * Default data source - uses IDC DICOMWeb directly. + * When ?gcp= param is present, the GCP extension activates the merge data source. */ - defaultDataSourceName: 'idc-merge', + defaultDataSourceName: 'idc-dicomweb', dataSources: [ /**