Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

multi archives issue #39

Open
alcir opened this issue Nov 26, 2021 · 6 comments
Open

multi archives issue #39

alcir opened this issue Nov 26, 2021 · 6 comments

Comments

@alcir
Copy link

alcir commented Nov 26, 2021

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.

@nroduit
Copy link
Owner

nroduit commented Nov 27, 2021

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.

@alcir
Copy link
Author

alcir commented Nov 29, 2021

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
Copy link
Author

alcir commented Dec 20, 2021

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
Copy link
Author

alcir commented Dec 20, 2021

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
Copy link
Author

alcir commented Dec 20, 2021

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.

@nroduit
Copy link
Owner

nroduit commented Dec 20, 2021

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants