fix: add the quotation exclusion for accept header for our configs (#3006)

* fix: revert the quotation inclusion for accept header to be off by default

* changed all configs
This commit is contained in:
Alireza 2022-11-04 09:01:34 -04:00 committed by GitHub
parent 7dc9b3549f
commit 3d0d09e1bc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
16 changed files with 34 additions and 2 deletions

View File

@ -52,7 +52,7 @@ export default function initWADOImageLoader(
// we should set this flag to false.
convertFloatPixelDataToInt: false,
},
beforeSend: function (xhr) {
beforeSend: function(xhr) {
const headers = UserAuthenticationService.getAuthorizationHeader();
// Request:
@ -61,7 +61,9 @@ export default function initWADOImageLoader(
// For now we use image/jls and image/x-jls because some servers still use the old type
// http://dicom.nema.org/medical/dicom/current/output/html/part18.html
const xhrRequestHeaders = {
Accept: (appConfig.ommitQuotationFromMultipartRequest) ? 'multipart/related; type=application/octet-stream' : 'multipart/related; type="application/octet-stream"'
Accept: appConfig.omitQuotationForMultipartRequest
? 'multipart/related; type=application/octet-stream'
: 'multipart/related; type="application/octet-stream"',
// 'multipart/related; type="image/x-jls", multipart/related; type="image/jls"; transfer-syntax="1.2.840.10008.1.2.4.80", multipart/related; type="image/x-jls", multipart/related; type="application/octet-stream"; transfer-syntax=*',
};

View File

@ -4,6 +4,8 @@ window.config = {
extensions: [],
modes: [],
showStudyList: true,
// below flag is for performance reasons, but it might not work for all servers
omitQuotationForMultipartRequest: true,
// filterQueryParam: false,
dataSources: [
{

View File

@ -5,6 +5,8 @@ window.config = {
modes: [],
showStudyList: true,
maxNumberOfWebWorkers: 3,
// below flag is for performance reasons, but it might not work for all servers
omitQuotationForMultipartRequest: true,
maxNumRequests: {
interaction: 100,
thumbnail: 75,

View File

@ -2,6 +2,8 @@ window.config = {
routerBasename: '/',
extensions: [],
showStudyList: true,
// below flag is for performance reasons, but it might not work for all servers
omitQuotationForMultipartRequest: true,
servers: {
dicomWeb: [
{

View File

@ -4,6 +4,8 @@ window.config = {
extensions: [],
modes: [],
showStudyList: true,
// below flag is for performance reasons, but it might not work for all servers
omitQuotationForMultipartRequest: true,
// filterQueryParam: false,
dataSources: [
{

View File

@ -5,6 +5,8 @@ window.config = {
modes: [],
showStudyList: true,
maxNumberOfWebWorkers: 3,
// below flag is for performance reasons, but it might not work for all servers
omitQuotationForMultipartRequest: true,
// filterQueryParam: false,
dataSources: [
{

View File

@ -3,6 +3,8 @@ window.config = {
showStudyList: true,
extensions: [],
modes: [],
// below flag is for performance reasons, but it might not work for all servers
omitQuotationForMultipartRequest: true,
dataSources: [
{
friendlyName: 'Orthanc Server',

View File

@ -1,6 +1,8 @@
window.config = {
routerBasename: '/',
showStudyList: true,
// below flag is for performance reasons, but it might not work for all servers
omitQuotationForMultipartRequest: true,
servers: {
// This is an array, but we'll only use the first entry for now
dicomWeb: [

View File

@ -3,6 +3,8 @@ window.config = {
showStudyList: true,
extensions: [],
modes: [],
// below flag is for performance reasons, but it might not work for all servers
omitQuotationForMultipartRequest: true,
dataSources: [
{
friendlyName: 'Orthanc Server',

View File

@ -4,6 +4,8 @@ window.config = {
extensions: [],
modes: [],
showStudyList: true,
// below flag is for performance reasons, but it might not work for all servers
omitQuotationForMultipartRequest: true,
// filterQueryParam: false,
dataSources: [
{

View File

@ -1,6 +1,8 @@
window.config = {
routerBasename: '/',
enableGoogleCloudAdapter: false,
// below flag is for performance reasons, but it might not work for all servers
omitQuotationForMultipartRequest: true,
// This is an array, but we'll only use the first entry for now
oidc: [
{

View File

@ -1,6 +1,8 @@
window.config = {
routerBasename: '/',
enableGoogleCloudAdapter: true,
// below flag is for performance reasons, but it might not work for all servers
omitQuotationForMultipartRequest: true,
servers: {
// This is an array, but we'll only use the first entry for now
dicomWeb: [],

View File

@ -3,6 +3,8 @@ window.config = {
showStudyList: true,
extensions: [],
modes: [],
// below flag is for performance reasons, but it might not work for all servers
omitQuotationForMultipartRequest: true,
dataSources: [
{
friendlyName: 'DCM4CHEE Server',

View File

@ -5,6 +5,8 @@ window.config = {
modes: [],
showStudyList: true,
maxNumberOfWebWorkers: 3,
// below flag is for performance reasons, but it might not work for all servers
omitQuotationForMultipartRequest: true,
// filterQueryParam: false,
dataSources: [
{

View File

@ -4,6 +4,8 @@ window.config = {
extensions: [],
modes: [],
showStudyList: true,
// below flag is for performance reasons, but it might not work for all servers
omitQuotationForMultipartRequest: true,
// filterQueryParam: false,
dataSources: [
{

View File

@ -1,6 +1,8 @@
window.config = {
routerBasename: '/',
showStudyList: true,
// below flag is for performance reasons, but it might not work for all servers
omitQuotationForMultipartRequest: true,
servers: {
dicomWeb: [
{