Skip to content

Commit 7443492

Browse files
authored
Merge pull request #2501 from hongwei1/develop
refactor/use in memory cache for createLocalisedResourceDocJsonCached…
2 parents 9c6844a + 3169ab0 commit 7443492

File tree

62 files changed

+160
-11967
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

62 files changed

+160
-11967
lines changed

.github/workflows/build_container_develop_branch.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -69,9 +69,6 @@ jobs:
6969
echo COUNTERPARTY_OTP_INSTRUCTION_TRANSPORT=dummy >> obp-api/src/main/resources/props/test.default.props
7070
echo SEPA_CREDIT_TRANSFERS_OTP_INSTRUCTION_TRANSPORT=dummy >> obp-api/src/main/resources/props/test.default.props
7171
72-
echo kafka.akka.timeout = 9 >> obp-api/src/main/resources/props/test.default.props
73-
echo remotedata.timeout = 10 >> obp-api/src/main/resources/props/test.default.props
74-
7572
echo allow_oauth2_login=true >> obp-api/src/main/resources/props/test.default.props
7673
echo oauth2.jwk_set.url=https://www.googleapis.com/oauth2/v3/certs >> obp-api/src/main/resources/props/test.default.props
7774

.github/workflows/build_container_non_develop_branch.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -69,9 +69,6 @@ jobs:
6969
echo COUNTERPARTY_OTP_INSTRUCTION_TRANSPORT=dummy >> obp-api/src/main/resources/props/test.default.props
7070
echo SEPA_CREDIT_TRANSFERS_OTP_INSTRUCTION_TRANSPORT=dummy >> obp-api/src/main/resources/props/test.default.props
7171
72-
echo kafka.akka.timeout = 9 >> obp-api/src/main/resources/props/test.default.props
73-
echo remotedata.timeout = 10 >> obp-api/src/main/resources/props/test.default.props
74-
7572
echo allow_oauth2_login=true >> obp-api/src/main/resources/props/test.default.props
7673
echo oauth2.jwk_set.url=https://www.googleapis.com/oauth2/v3/certs >> obp-api/src/main/resources/props/test.default.props
7774

.github/workflows/build_pull_request.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,6 @@ jobs:
6565
echo COUNTERPARTY_OTP_INSTRUCTION_TRANSPORT=dummy >> obp-api/src/main/resources/props/test.default.props
6666
echo SEPA_CREDIT_TRANSFERS_OTP_INSTRUCTION_TRANSPORT=dummy >> obp-api/src/main/resources/props/test.default.props
6767
68-
echo kafka.akka.timeout = 9 >> obp-api/src/main/resources/props/test.default.props
69-
echo remotedata.timeout = 10 >> obp-api/src/main/resources/props/test.default.props
7068
7169
echo allow_oauth2_login=true >> obp-api/src/main/resources/props/test.default.props
7270
echo oauth2.jwk_set.url=https://www.googleapis.com/oauth2/v3/certs >> obp-api/src/main/resources/props/test.default.props

README.kafka.md

Lines changed: 0 additions & 26 deletions
This file was deleted.

README.md

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -374,17 +374,6 @@ We use 9 to run the API in production mode.
374374

375375
Most internal OBP model data access now occurs over Akka. This is so the machine that has JDBC access to the OBP database can be physically separated from the OBP API layer. In this configuration we run two instances of OBP-API on two different machines and they communicate over Akka. Please see README.Akka.md for instructions.
376376

377-
## Using SSL Encryption with kafka
378-
379-
For SSL encryption we use JKS keystores. Note that both the keystore and the truststore (and all keys within) must have the same password for unlocking, for which the API will stop at boot up and ask for.
380-
381-
* Edit your props file(s) to contain:
382-
383-
```
384-
kafka.use.ssl=true
385-
keystore.path=/path/to/api.keystore.jks
386-
truststore.path=/path/to/api.truststore.jks
387-
```
388377

389378
## Using SSL Encryption with RabbitMq
390379

cheat_sheet.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,6 @@
2424

2525
[Access Control](https://apiexplorersandbox.openbankproject.com/glossary#API.Access-Control)
2626

27-
[OBP Kafka](https://apiexplorersandbox.openbankproject.com/glossary#Adapter.Kafka.Intro)
28-
2927
[OBP Akka](https://apiexplorersandbox.openbankproject.com/glossary#Adapter.Akka.Intro)
3028

3129
[API Explorer](https://github.com/OpenBankProject/API-Explorer/blob/develop/README.md)

completed_developments.md

Lines changed: 1 addition & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -278,10 +278,6 @@ Support for on premise OAuth2 provider e.g. MitreId. See the glossary.
278278
### Message Docs (for Akka)
279279
Message Docs (which define Core Banking System Akka messages) are now available independent of the connector being used on the API instance. See [here](https://apiexplorersandbox.openbankproject.com/?ignoredefcat=true&tags=#v2_2_0-getMessageDocs)
280280

281-
282-
### Message Docs (for Kafka)
283-
Message Docs (which define Core Banking System Kafka messages) are now available independent of the connector being used on the API instance. See [here](https://apiexplorersandbox.openbankproject.com/?ignoredefcat=true&tags=#v2_2_0-getMessageDocs)
284-
285281
### Endpoint config and cleanup
286282
Endpoints can now be enabled / disabled explicitly using Props file.
287283
We removed old versions including v1.0, v1.1 and v.1.2.
@@ -302,15 +298,11 @@ We added Custom code folders so that bank specific forks can more easily git mer
302298

303299

304300
### API Tester
305-
API Tester is a Python/Djano App for testing an OBP API instance from the outside. Partiularly useful when using a non-sandbox (e.g. kafka) connector. It supports a variety of authentication methods so you can test outside a gateway. You can configure different data profiles for specifying parameters such as bank_id, account_id etc. See [here](https://github.com/OpenBankProject/API-Tester) for the source code and installation instructions.
301+
API Tester is a Python/Djano App for testing an OBP API instance from the outside. Partiularly useful when using a non-sandbox (e.g. RabbitMq) connector. It supports a variety of authentication methods so you can test outside a gateway. You can configure different data profiles for specifying parameters such as bank_id, account_id etc. See [here](https://github.com/OpenBankProject/API-Tester) for the source code and installation instructions.
306302

307303
### Extend Swagger support
308304
We improved the information contained in the Swagger (and Resource Doc) endpoints. They are also available from the API Explorer. See [here](https://apiexplorersandbox.openbankproject.com/?ignoredefcat=true&tags=#v1_4_0-getResourceDocsSwagger)
309305

310-
311-
### Kafka versioning
312-
The built in kafka connectors now provide message versioning
313-
314306
### Akka Remote data (Three tier architechture)
315307
Most OBP data access now happens over Akka. This allows the API layer to be physically separated from the storage layer with the API layer only able to call a specified set of data access functions with only the storage layer having JDBC / SQL access.
316308

@@ -428,13 +420,6 @@ is used to explore and interact with the OBP API. See [API Explorer on Sandbox](
428420
See [Resource Docs endpoint](https://api.openbankproject.com/obp/v1.4.0/resource-docs/obp)
429421

430422

431-
### Kafka connector
432-
433-
* Get transactions via Kafka bus and language neutral connector on the south side of the MQ
434-
435-
See [Docker obp-full-kafka](https://hub.docker.com/r/openbankproject/obp-full-kafka/)
436-
437-
438423
### Version 1.4.0
439424

440425
This version is stable. For the spec see [here](https://github.com/OpenBankProject/OBP-API/wiki/REST-API-V1.4.0) or [here](https://apiexplorersandbox.openbankproject.com/?version=1.4.0&list-all-banks=false&core=&psd2=&obwg=&ignoredefcat=true)

obp-api/pom.xml

Lines changed: 0 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -79,16 +79,6 @@
7979
<groupId>org.slf4j</groupId>
8080
<version>1.7.26</version>
8181
</dependency>
82-
<dependency>
83-
<groupId>org.apache.kafka</groupId>
84-
<artifactId>kafka-clients</artifactId>
85-
<version>${kafka.version}</version>
86-
</dependency>
87-
<dependency>
88-
<groupId>org.apache.kafka</groupId>
89-
<artifactId>connect-json</artifactId>
90-
<version>${kafka.version}</version>
91-
</dependency>
9282
<dependency>
9383
<groupId>org.bouncycastle</groupId>
9484
<artifactId>bcpg-jdk15on</artifactId>
@@ -253,11 +243,6 @@
253243
<artifactId>akka-remote_${scala.version}</artifactId>
254244
<version>${akka.version}</version>
255245
</dependency>
256-
<dependency>
257-
<groupId>com.typesafe.akka</groupId>
258-
<artifactId>akka-stream-kafka_${scala.version}</artifactId>
259-
<version>${akka-streams-kafka.version}</version>
260-
</dependency>
261246
<dependency>
262247
<groupId>com.sksamuel.avro4s</groupId>
263248
<artifactId>avro4s-core_${scala.version}</artifactId>
@@ -512,14 +497,6 @@
512497
<version>1.20.3</version>
513498
<scope>test</scope>
514499
</dependency>
515-
<!-- https://mvnrepository.com/artifact/org.testcontainers/kafka -->
516-
<dependency>
517-
<groupId>org.testcontainers</groupId>
518-
<artifactId>kafka</artifactId>
519-
<version>1.20.3</version>
520-
<scope>test</scope>
521-
</dependency>
522-
523500
</dependencies>
524501

525502
<build>

obp-api/src/main/resources/i18n/lift-core_es_ES.properties

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -947,15 +947,11 @@ Este error no puede ser mostrado al usuario, sólo para depuración.
947947
OBP-50004 = método (AuthUser.getCurrentUser) no puede encontrar el usuario actual en el contexto actual!
948948
OBP-50005 = ha producido un error interno o no especificado.
949949
OBP-50006 = interrumpió la excepción.
950-
OBP-50007 = de ejecución de Kafka.
951-
OBP-50008 = de tiempo de espera del flujo Kafka de Akka.
952-
OBP-50009 = desconocido de Kafka.
953950
OBP-50010 = devuelve la caja vacía a Liftweb.
954951
OBP-50012 = se puede obtener el objeto CallContext aquí.
955952
OBP-50013 = sistema bancario central devolvió un error o una respuesta no especificada.
956953
OBP-50014 = se puede actualizar el usuario.
957954
OBP-50015 = servidor encontró una condición inesperada que le impidió cumplir con la solicitud.
958-
OBP-50016 = servidor kafka no está disponible.
959955
OBP-50017 = punto final está prohibido en esta instancia de la API.
960956
OBP-50018 = de construcción.
961957
OBP-50019 = se puede conectar a la base de datos OBP.
@@ -981,7 +977,6 @@ OBP-50217 = no devolvió la transacción que solicitamos.
981977
OBP-50218 = conector no devolvió el conjunto de etiquetas de punto final que solicitamos.
982978
OBP-50219 = no devolvió las cuentas bancarias que solicitamos.
983979
#Excepciones del adaptador (OBP-6XXXX)
984-
#Reservado para mensajes del adaptador (al sur de Kafka)
985980
#También se utiliza para el conector == mapeado, y mostrarlo como los errores internos.
986981
OBP-60001 = excepción de transacción.
987982
OBP-60002 = la Excepción de Valor de Carga.

obp-api/src/main/resources/props/sample.props.template

Lines changed: 2 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ connector=star
3131
#hikari.keepaliveTime=
3232
#hikari.maxLifetime=
3333

34-
## if connector = star, then need to set which connectors will be used. For now, obp support rest, akka, kafka. If you set kafka, then you need to start the kafka server.
34+
## if connector = star, then need to set which connectors will be used. For now, obp support rest, akka.
3535
starConnector_supported_types=mapped,internal
3636

3737
## whether export LocalMappedConnector methods as endpoints, it is just for develop, default is false
@@ -125,34 +125,13 @@ long_endpoint_timeout = 55000
125125
## Scheduler will be disabled if delay is not set.
126126
#transaction_status_scheduler_delay=300
127127

128-
## If using kafka, set the brokers
129-
#kafka.bootstrap_hosts=localhost:9092
130-
# WARNING: if this number does not match the partitions in Kafka config, you will SUFFER !
131-
#kafka.partitions=3
132-
133-
#This is the api instance, we create kafka topic based on this number, each instance should have each own id. use it in load balancing + Kafka setup
134-
#This is also used for scheduler.
135-
#OBP set the default as the non-persistent UUID string.
136-
#api_instance_id=7uy8a7e4-6d02-40e3-a129-0b2bf89de8uh
137-
#If a value is set, OBP will concatenate the UUID string with the value.
138-
#api_instance_id=1
139-
#When the value is set to conclude with "final," OBP will preserve that value.
140-
#api_instance_id=1_final
141-
142-
## DEPRECATED
143-
## Enable user authentication via kafka
144-
#kafka.user.authentication=true
145-
146128

147129
## Enable user authentication via the connector
148130
#connector.user.authentication=true
149131

150132
## Enable SSL for JWT, if set to true must set paths for the keystore locations
151133
jwt.use.ssl=false
152134

153-
## Enable SSL for kafka, if set to true must set paths for the keystore locations
154-
#kafka.use.ssl=true
155-
156135
## Enable SSL for rabbitmq, if set to true must set paths for the keystore locations
157136
#rabbitmq.use.ssl=false
158137

@@ -710,9 +689,6 @@ autocomplete_at_login_form_enabled=false
710689
# To BYPASS this security features (for local development only), set this property to true to skip the email address validation.
711690
#authUser.skipEmailValidation=false
712691

713-
# If using Kafka but want to get counterparties from OBP, set this to true
714-
#get_counterparties_from_OBP_DB=true
715-
716692
# control the create and access to public views.
717693
# allow_public_views=false
718694

@@ -756,7 +732,7 @@ dauth.host=127.0.0.1
756732
# }
757733
# When is enabled we show all messages in a chain. For instance:
758734
# {
759-
# "error": "OBP-30001: Bank not found. Please specify a valid value for BANK_ID. <- Full(Kafka_TimeoutExceptionjava.util.concurrent.TimeoutException: The stream has not been completed in 1550 milliseconds.)"
735+
# "error": "OBP-30001: Bank not found. Please specify a valid value for BANK_ID. <- Full(TimeoutExceptionjava.util.concurrent.TimeoutException: The stream has not been completed in 1550 milliseconds.)"
760736
# }
761737
display_internal_errors=false
762738
# -------------------------------------- Display internal errors --

0 commit comments

Comments
 (0)