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

Issue/239 port 1.6.0 to 2.0.0 #244

Merged
merged 35 commits into from
Oct 15, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
036328e
fix for #224
hhund Sep 4, 2024
32241f1
Merge remote-tracking branch
hhund Sep 4, 2024
8064f56
New OrganizationAffiliation role change test case
hhund Oct 2, 2024
cc7588a
Performance tests for Organization and OrganizationAffiliation updates
hhund Oct 2, 2024
5179b8e
code formatting
hhund Oct 2, 2024
2833ce1
version to 1.6.0-SNAPSHOT
hhund Oct 2, 2024
4b8a60c
new all_read_access_resources view and improved trigger
hhund Oct 2, 2024
4f46d85
improved constant name
hhund Oct 2, 2024
8b12fcb
fixed length checks (copy/paste error)
hhund Oct 2, 2024
c61d35d
added missing static and final modifiers
hhund Oct 2, 2024
29d5781
added missing negation to correct logic error
hhund Oct 7, 2024
54b5a3a
nginx config cleanup and version upgrade
hhund Oct 7, 2024
b2072d8
forward proxy image and service, proxy configured for dic1-bpe
hhund Oct 7, 2024
ef240cb
code cleanup
hhund Oct 7, 2024
5ee060b
enables reconnect including REST GET for abnormally closed websockets
hhund Oct 7, 2024
bc48b41
Added integration test for posting a Questionnaire and a corresponding
jaboehri Sep 5, 2024
468d460
No idea why I needed those in the first place
jaboehri Sep 5, 2024
76cda1b
Asserts all resource have been created
jaboehri Sep 5, 2024
361d360
Removed setting resource id's unnecessarily. Removed unused imports.
jaboehri Sep 6, 2024
2650244
partial fix for #226
hhund Sep 9, 2024
0094c2a
moved questionnaire canonical reference check, code cleanup, more tests
hhund Sep 30, 2024
26fb156
remove not needed parameter from private method
hhund Oct 2, 2024
990f94d
dependency and maven plugin version upgrades
hhund Oct 8, 2024
7be791c
Merge branch
hhund Oct 8, 2024
a9ee855
Merge branch 'issue/230_Org_OrgAff_Update_Performance' into develop
hhund Oct 8, 2024
be96436
Merge remote-tracking branch 'origin/issue/232_OIDC_Provider_No_Proxy'
hhund Oct 8, 2024
ed83bac
Merge remote-tracking branch
hhund Oct 8, 2024
eeddda2
Merge remote-tracking branch 'origin/develop' into
hhund Oct 8, 2024
04b6829
Merge remote-tracking branch 'origin/issue/236_Upgrade_Dependencies'
hhund Oct 9, 2024
b7e7419
set level DEBUG for 'de.dkfz.nct' in default bpe log config
hhund Oct 14, 2024
a462213
upgraded keycloak in 3dic/ttp test setup: 21.0 -> 25.0
hhund Oct 14, 2024
68e6ba9
added allowlist management publication
hhund Oct 15, 2024
e3c59c5
1.6.0 release
hhund Oct 15, 2024
f761d2d
Merge remote-tracking branch 'origin/release/1.6.0' into main
hhund Oct 15, 2024
c954bc5
Merge remote-tracking branch 'origin/main' into
hhund Oct 15, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 21 additions & 1 deletion CITATION.cff
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ preferred-citation:
type: proceedings
title: "Data Sharing Framework (DSF)"
version: 2.0.0
date-released: 2024-07-15
date-released: 2024-10-15
url: https://dsf.dev
repository-code: https://github.com/datasharingframework/dsf
repository-artifact: https://github.com/datasharingframework/dsf/releases
Expand Down Expand Up @@ -241,3 +241,23 @@ references:
year: 2024
pages: 28-32
doi: 10.3233/SHTI230921
- type: proceedings
authors:
- family-names: Schweizer
given-names: Simon Tobias
- family-names: Hund
given-names: Hauke
- family-names: Kurscheidt
given-names: Maximilian
- family-names: Zilske
given-names: Christoph
- family-names: Böhringer
given-names: Jan P.
- family-names: Fegeler
given-names: Christian
title: "Handling Complexity in Decentralized Research Networks: The Data Sharing Framework Allowlist Management Application"
journal: Stud Health Technol Inform
volume: 317
year: 2024
pages: 85-93
doi: 10.3233/SHTI240841
2 changes: 1 addition & 1 deletion dsf-bpe/dsf-bpe-server-jetty/conf/log4j2.xml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<Configuration status="INFO" monitorInterval="30"
<Configuration status="WARN" monitorInterval="30"
verbose="false">

<Appenders>
Expand Down
3 changes: 2 additions & 1 deletion dsf-bpe/dsf-bpe-server-jetty/docker/conf/log4j2.xml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<Configuration status="INFO" monitorInterval="30" verbose="false">
<Configuration status="WARN" monitorInterval="30" verbose="false">
<Appenders>
<Console name="CONSOLE" target="SYSTEM_OUT">
<PatternLayout pattern="%highlight{%p %t - %C{1}.%M(%L) | %m}{FATAL=red, ERROR=red, WARN=yellow, INFO=white, DEBUG=white, TRACE=white}%n" />
Expand All @@ -17,6 +17,7 @@
<Logger name="dev.dsf" level="DEBUG" />
<Logger name="de.netzwerk_universitaetsmedizin" level="DEBUG" />
<Logger name="de.medizininformatik_initiative" level="DEBUG" />
<Logger name="de.dkfz.nct" level="DEBUG" />
<Logger name="org.eclipse.jetty" level="INFO" />
<Logger name="ca.uhn.fhir.parser.LenientErrorHandler" level="ERROR" />

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,17 +11,10 @@ public MultiVersionSpringProcessEngineConfiguration(DelegateProvider delegatePro
}

@Override
protected void initTelemetry()
protected void initDiagnostics()
{
// override to turn telemetry collection of
// see also CamundaConfig
}

@Override
public TelemetryDataImpl getTelemetryData()
{
// NPE fix after turning off telemetry collection
// see also CamundaConfig
return new TelemetryDataImpl(null, null);
setTelemetryData(new TelemetryDataImpl(null, null));
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -99,10 +99,6 @@ public SpringProcessEngineConfiguration processEngineConfiguration()
processPluginFactories.stream().flatMap(ProcessPluginFactory::getSerializer).toList());
c.setFallbackSerializerFactory(fallbackSerializerFactory());

// see also MultiVersionSpringProcessEngineConfiguration
c.setInitializeTelemetry(false);
c.setTelemetryReporterActivate(false);

DefaultJobExecutor jobExecutor = new DefaultJobExecutor();
jobExecutor.setCorePoolSize(propertiesConfig.getProcessEngineJobExecutorCorePoolSize());
jobExecutor.setQueueSize(propertiesConfig.getProcessEngineJobExecutorQueueSize());
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -228,7 +228,7 @@ private void connectWebsocket(Subscription subscription)
{
try
{
WebsocketClient client = clientProvider.getLocalWebsocketClient(() -> connect(),
WebsocketClient client = clientProvider.getLocalWebsocketClient(this::connect,
subscription.getIdElement().getIdPart());

EventType eventType = toEventType(subscription.getChannel().getPayload());
Expand Down Expand Up @@ -260,7 +260,7 @@ private Void onError(Throwable t)
{
// no debug log, exception previously logged by retrieveWebsocketSubscription, loadNewResources and
// connectWebsocket methods
logger.error("Error while loading existing {} resources and connecting websocket: {} - {}", resourceName,
logger.error("Error loading existing {} resources and connecting websocket: {} - {}", resourceName,
t.getClass().getName(), t.getMessage());

return null;
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
--
-- Copyright Camunda Services GmbH and/or licensed to Camunda Services GmbH
-- under one or more contributor license agreements. See the NOTICE file
-- distributed with this work for additional information regarding copyright
-- ownership. Camunda licenses this file to you under the Apache License,
-- Version 2.0; you may not use this file except in compliance with the License.
-- You may obtain a copy of the License at
--
-- http://www.apache.org/licenses/LICENSE-2.0
--
-- Unless required by applicable law or agreed to in writing, software
-- distributed under the License is distributed on an "AS IS" BASIS,
-- WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-- See the License for the specific language governing permissions and
-- limitations under the License.
--

insert into ACT_GE_SCHEMA_LOG
values ('1100', CURRENT_TIMESTAMP, '7.22.0');

alter table ACT_RU_TASK add column TASK_STATE_ varchar(64);

alter table ACT_HI_TASKINST add column TASK_STATE_ varchar(64);

alter table ACT_RU_JOB add column BATCH_ID_ varchar(64);
alter table ACT_HI_JOB_LOG add column BATCH_ID_ varchar(64);

alter table ACT_HI_PROCINST add RESTARTED_PROC_INST_ID_ varchar(64);
create index ACT_IDX_HI_PRO_RST_PRO_INST_ID on ACT_HI_PROCINST(RESTARTED_PROC_INST_ID_);
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
<?xml version="1.0" encoding="UTF-8"?>
<databaseChangeLog
xmlns="http://www.liquibase.org/xml/ns/dbchangelog"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://www.liquibase.org/xml/ns/dbchangelog
http://www.liquibase.org/xml/ns/dbchangelog/dbchangelog-3.6.xsd">

<changeSet author="camunda.org" id="db.camunda_engine.changelog-1.6.0">
<sqlFile dbms="postgresql" encoding="utf8" path="db/camunda/postgres_engine_7.21_to_7.22.sql" />
</changeSet>

</databaseChangeLog>
2 changes: 2 additions & 0 deletions dsf-bpe/dsf-bpe-server/src/main/resources/db/db.changelog.xml
Original file line number Diff line number Diff line change
Expand Up @@ -17,4 +17,6 @@
<include file="db/db.camunda_engine.changelog-1.3.0.xml" />

<include file="db/db.camunda_engine.changelog-1.5.1.xml" />

<include file="db/db.camunda_engine.changelog-1.6.0.xml" />
</databaseChangeLog>
2 changes: 1 addition & 1 deletion dsf-bpe/dsf-bpe-server/src/test/resources/log4j2.xml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<Configuration status="INFO" monitorInterval="30"
<Configuration status="WARN" monitorInterval="30"
verbose="false">

<Appenders>
Expand Down
2 changes: 1 addition & 1 deletion dsf-common/dsf-common-auth/src/test/resources/log4j2.xml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<Configuration status="INFO" monitorInterval="30" verbose="false">
<Configuration status="WARN" monitorInterval="30" verbose="false">

<Appenders>
<Console name="CONSOLE" target="SYSTEM_OUT">
Expand Down
2 changes: 1 addition & 1 deletion dsf-common/dsf-common-config/src/test/resources/log4j2.xml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<Configuration status="INFO" monitorInterval="30" verbose="false">
<Configuration status="WARN" monitorInterval="30" verbose="false">

<Appenders>
<Console name="CONSOLE" target="SYSTEM_OUT">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -431,7 +431,7 @@ private Duration oidcClientConnectTimeout()
private Proxy oidcClientProxy()
{
ProxyConfig config = new ProxyConfigImpl(proxyUrl, proxyUsername, proxyPassword, proxyNoProxy);
if (config.getUrl() != null && config.isNoProxyUrl(oidcProviderRealmBaseUrl))
if (config.getUrl() != null && !config.isNoProxyUrl(oidcProviderRealmBaseUrl))
{
try
{
Expand Down
2 changes: 1 addition & 1 deletion dsf-common/dsf-common-jetty/src/test/resources/log4j2.xml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<Configuration status="INFO" monitorInterval="30" verbose="false">
<Configuration status="WARN" monitorInterval="30" verbose="false">

<Appenders>
<Console name="CONSOLE" target="SYSTEM_OUT">
Expand Down
22 changes: 19 additions & 3 deletions dsf-docker-test-setup-3dic-ttp/docker-compose.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
version: '3.8'
services:
proxy:
image: nginx:1.23
image: nginx:1.27
restart: "no"
ports:
- 127.0.0.1:443:443
Expand Down Expand Up @@ -78,7 +78,7 @@ services:
read_only: true

keycloak:
image: quay.io/keycloak/keycloak:21.0
image: quay.io/keycloak/keycloak:25.0
restart: "no"
ports:
- 127.0.0.1:8443:8443
Expand All @@ -104,7 +104,16 @@ services:
--https-certificate-key-file=/run/secrets/keycloak_certificate_private_key.pem
--spi-truststore-file-file=/run/secrets/keycloak_trust_store.jks
--spi-truststore-file-password=password
--spi-truststore-file-hostname-verification-policy=STRICT
--spi-truststore-file-hostname-verification-policy=DEFAULT

forward-proxy:
build: ./forward-proxy
restart: "no"
environment:
TZ: Europe/Berlin
networks:
forward-proxy:
internet:

dic1-fhir:
build: ../dsf-fhir/dsf-fhir-server-jetty/docker
Expand Down Expand Up @@ -518,15 +527,21 @@ services:
DEV_DSF_SERVER_AUTH_OIDC_PROVIDER_CLIENT_TRUST_SERVER_CERTIFICATE_CAS: /run/secrets/app_server_trust_certificates.pem
DEV_DSF_SERVER_AUTH_OIDC_CLIENT_ID: dic1-bpe
DEV_DSF_SERVER_AUTH_OIDC_CLIENT_SECRET: ytqFCErw9GfhVUrrM8xc0Grbu4r7qGig
DEV_DSF_PROXY_URL: http://forward-proxy:8080
DEV_DSF_PROXY_USERNAME: proxy_user
DEV_DSF_PROXY_PASSWORD: proxy_password
DEV_DSF_PROXY_NOPROXY: keycloak
networks:
dic1-bpe-frontend:
ipv4_address: 172.20.0.35
dic1-bpe-backend:
internet:
forward-proxy:
depends_on:
- db
- dic1-fhir
- keycloak
- forward-proxy

dic2-bpe:
build: ../dsf-bpe/dsf-bpe-server-jetty/docker
Expand Down Expand Up @@ -944,6 +959,7 @@ networks:
- subnet: 172.20.0.56/29
ttp-bpe-backend:
internet:
forward-proxy:

volumes:
db-data:
Expand Down
4 changes: 4 additions & 0 deletions dsf-docker-test-setup-3dic-ttp/forward-proxy/Dockerfile
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
FROM alpine:3.20
RUN apk add --no-cache tinyproxy
COPY tinyproxy.conf /etc/tinyproxy/tinyproxy.conf
CMD ["tinyproxy", "-d"]
6 changes: 6 additions & 0 deletions dsf-docker-test-setup-3dic-ttp/forward-proxy/tinyproxy.conf
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
User tinyproxy
Group tinyproxy
Port 8080
Timeout 100
LogLevel Connect
BasicAuth proxy_user proxy_password
46 changes: 0 additions & 46 deletions dsf-docker-test-setup-3dic-ttp/proxy/conf.d/default.conf

This file was deleted.

21 changes: 1 addition & 20 deletions dsf-docker-test-setup-3dic-ttp/proxy/nginx.conf
Original file line number Diff line number Diff line change
@@ -1,16 +1,6 @@

user nginx;
worker_processes 1;

error_log /var/log/nginx/error.log warn;
pid /var/run/nginx.pid;


events {
worker_connections 1024;
}


http {
include /etc/nginx/mime.types;
default_type application/octet-stream;
Expand All @@ -19,15 +9,6 @@ http {
'$status $body_bytes_sent "$http_referer" '
'"$http_user_agent" "$http_x_forwarded_for"';

access_log /var/log/nginx/access.log main;

sendfile on;
#tcp_nopush on;

keepalive_timeout 65;

#gzip on;

ssl_certificate /run/secrets/proxy_certificate_and_int_cas.pem;
ssl_certificate_key /run/secrets/proxy_certificate_private_key.pem;
ssl_protocols TLSv1.2 TLSv1.3;
Expand All @@ -45,4 +26,4 @@ http {
}

include /etc/nginx/conf.d/*.conf;
}
}
2 changes: 1 addition & 1 deletion dsf-fhir/dsf-fhir-auth/src/test/resources/log4j2.xml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<Configuration status="INFO" monitorInterval="30" verbose="false">
<Configuration status="WARN" monitorInterval="30" verbose="false">

<Appenders>
<Console name="CONSOLE" target="SYSTEM_OUT">
Expand Down
Loading
Loading