Skip to content

Commit

Permalink
Merge pull request #4 from OSGP/feature/FDP-94
Browse files Browse the repository at this point in the history
FDP-94: Make Kafka group id configurable
  • Loading branch information
jasperkamerling authored Nov 22, 2023
2 parents 382dbb4 + 5654742 commit a645dd5
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 3 deletions.
2 changes: 2 additions & 0 deletions application/src/integrationTest/resources/application.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
spring:
kafka:
bootstrap-servers: ${spring.embedded.kafka.brokers}
consumer:
group-id: gxf-soap-bridge
server:
ssl:
enabled: true
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,8 @@ class ProxyRequestKafkaListener(private val platformCommunicationService: Platfo
@KafkaListener(
id = "gxf-request-consumer",
topics = ["\${kafka.incoming.requests.topic}"],
concurrency = "\${kafka.incoming.requests.concurrency}"
concurrency = "\${kafka.incoming.requests.concurrency}",
idIsGroup = false
)
fun consume(record: ConsumerRecord<String, String>) {
logger.debug { "Received request: ${record.key()}, ${record.value()}" }
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,8 @@ class ProxyResponseKafkaListener(private val clientCommunicationService: ClientC
@KafkaListener(
id = "gxf-response-consumer",
topics = ["\${kafka.incoming.responses.topic}"],
concurrency = "\${kafka.incoming.responses.concurrency}"
concurrency = "\${kafka.incoming.responses.concurrency}",
idIsGroup = false
)
fun consume(record: ConsumerRecord<String, String>) {
logger.debug { "Received response: ${record.key()}, ${record.value()}" }
Expand Down
7 changes: 6 additions & 1 deletion application/src/main/resources/application.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,11 @@ management:
exposure:
include: prometheus

spring:
kafka:
consumer:
group-id: gxf-soap-bridge

soap:
hostname-verification-strategy: BROWSER_COMPATIBLE_HOSTNAMES

Expand All @@ -12,4 +17,4 @@ kafka:
requests:
concurrency: 1
responses:
concurrency: 1
concurrency: 1

0 comments on commit a645dd5

Please sign in to comment.