weasis-pacs-connector icon indicating copy to clipboard operation
weasis-pacs-connector copied to clipboard

multi archives issue

Open alcir opened this issue 2 years ago • 6 comments

Something has changed between 7.1.1 and 7.1.2? With the same configuration the behavior is different: in 7.1.1 it works, in 7.1.2 doesn't work anymore.

arc.config.list=arc1,arc2 in weasis-pacs-connector.properties (and corresponding arc1 and arc2 files), it seems that they are not taken in consideration and the baseUrl= indeed contains the url of the server where weasis-pacs-connector is deployed, instead of arc.wado.url= specified in the archive properties files.

alcir avatar Nov 26 '21 11:11 alcir

Please provide a description of the context and a reproducible use case. What are the errors.

baseUrl is not a property and arc.wado.url is the Base URL of the WADO server (for downloading image). Check the configuration instructions.

nroduit avatar Nov 27 '21 16:11 nroduit

Hello. I mean. The same weasis-pacs-connector.properties file, containing arc.config.list=arc1,arc2, and the corresponding arc1 and arc2 configuration files, with 7.1.1 works, while with 7.1.2 it doesn't work.

arc1 configuration file contains arc.wado.url=http://arc1:8080/wado while arc2 contains arc.wado.url=http://arc2:8080/wado

weasis-pacs-connector is installed on a third server (that is not arc1 or arc2).

If I invoke, for instance, https://server.example/weasis-pacs-connector/manifest?accessionNumber=123.456 In the resulting file (with 7.1.2) I obtain <arcQuery arcId="1000" baseUrl="https://server.example/dcm4chee-arc/aets/DCM4CHEE/wado" requireOnlySOPInstanceUID="false" additionnalParameters="&amp;transferSyntax=*" > that is wrong.

While with 7.1.1 I correctly get <arcQuery arcId="1000" baseUrl="http://arc1:8080/wado" requireOnlySOPInstanceUID="false" >

Like if: or the arc.wado.url= in the properties file is not taken into account, or the files listed in arc.config.list= are not taken into account.

Where can I grab possible error messages?

alcir avatar Nov 29 '21 10:11 alcir

Check the configuration instructions.

So the point could be this one?

This file named weasis-pacs-connector.properties and dicom-dcm4chee.properties must be placed in the classpath of the application:
    ...
    In Tomcat just specify the directory in shared.loader property of /conf/catalina.properties

With 7.1.1, I placed these files in ${catalina.home}/lib and it was working. Now with 7.1.2 it doesn't take into account these files anymore.

alcir avatar Dec 20 '21 12:12 alcir

However, even defining the shared.loader property in /conf/catalina.properties, and placing weasis-pacs-connector.properties and dicom-dcm4chee.properties in there, these files are not taken into account.

alcir avatar Dec 20 '21 12:12 alcir

Well. While deploying 7.1.1, in the Tomcat log I can see

2021-12-20 13:43:41,521 [06-8080-exec-12] INFO  o.w.s.ManifestManager     External weasis-pacs-connector configuration: file:/opt/apache-tomcat-8.5.38/shared/weasis-pacs-connector.properties
2021-12-20 13:43:41,526 [06-8080-exec-12] INFO  o.w.s.ManifestManager     Archive configuration: file:/opt/apache-tomcat-8.5.38/shared/pacs1-dcm4chee.properties
2021-12-20 13:43:41,526 [06-8080-exec-12] INFO  o.w.s.ManifestManager     Archive configuration: file:/opt/apache-tomcat-8.5.38/shared/pacs2-dcm4chee.properties
2021-12-20 13:43:41,526 [06-8080-exec-12] INFO  o.w.s.ManifestManager     Archive configuration: file:/opt/apache-tomcat-8.5.38/shared/pacs2-dcm4chee.properties

(In this case, /opt/apache-tomcat-8.5.38/shared/ is the directory defined in shared.loader property of /conf/catalina.properties).

While deploying 7.1.2 i see

2021-12-20 13:47:57,103 [06-8080-exec-34] INFO  o.w.s.ManifestManager     Path of weasis-pacs-connector configuration: file:/opt/apache-tomcat-8.5.38/webapps/weasis-pacs-connector/WEB-INF/classes/weasis-pacs-connector.properties
2021-12-20 13:47:57,108 [06-8080-exec-34] INFO  o.w.s.ManifestManager     Archive configuration: file:/opt/apache-tomcat-8.5.38/webapps/weasis-pacs-connector/WEB-INF/classes/dicom-dcm4chee-arc.properties

And the Tomcat instance/configuration is the same.

alcir avatar Dec 20 '21 12:12 alcir

Did you try the build the package from the last commit?

There are several commits not published by me between 7.1.1 and 7.1.2. So I have to investigate.

I will try to make a new release next week.

nroduit avatar Dec 20 '21 17:12 nroduit