Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

Commit d8dbfdf

Browse files
authored
Merge pull request #1 from veshu/develop
fixes for topcoder-archive/topcoder-platform-topcoder-x-ui#21
2 parents 7e4bdb6 + 7202c43 commit d8dbfdf

File tree

5 files changed

+107
-197
lines changed

5 files changed

+107
-197
lines changed

config/default.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,8 @@ module.exports = {
1818
LOG_LEVEL: process.env.LOG_LEVEL || 'info',
1919
TOPIC: process.env.TOPIC || 'tc-x-events',
2020
KAFKA_OPTIONS: {
21-
kafkaHost: process.env.KAFKA_HOST || 'localhost:9092',
22-
sslOptions: {
21+
connectionString: process.env.KAFKA_HOST || 'localhost:9092',
22+
ssl: {
2323
cert: process.env.KAFKA_CLIENT_CERT || fs.readFileSync('./kafka_client.cer'), // eslint-disable-line no-sync
2424
key: process.env.KAFKA_CLIENT_CERT_KEY || fs.readFileSync('./kafka_client.key') // eslint-disable-line no-sync
2525
}

configuration.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,13 +13,13 @@ The following config parameters are supported, they are defined in `config/defau
1313
|KAFKA_CLIENT_CERT | The Kafka SSL certificate to use when connecting| Read from kafka_client.cer file, but this can be set as a string like it is on Heroku |
1414
|KAFKA_CLIENT_CERT_KEY | The Kafka SSL certificate key to use when connecting| Read from kafka_client.key file, but this can be set as a string like it is on Heroku|
1515

16-
KAFKA_OPTIONS should be object as described in https://github.com/SOHU-Co/kafka-node#kafkaclient
16+
KAFKA_OPTIONS should be object as described in https://github.com/oleksiyk/kafka#ssl
1717
For using with SSL, the options should be as
1818
```
1919
{
20-
kafkaHost: '<server>',
21-
sslOptions: {
22-
cert: '<certificate>',
20+
connectionString: '<server>',
21+
ssl: {
22+
cert: '<certificate>',
2323
key: '<key>'
2424
}
2525
}

0 commit comments

Comments
 (0)