Skip to content

Commit

Permalink
Reworked redis client connection logic
Browse files Browse the repository at this point in the history
  • Loading branch information
oklemenz2 committed Jan 8, 2025
1 parent 0f9f985 commit 4cb7667
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -341,7 +341,7 @@ over remote distribution via Redis.

### Authentication & Authorization

Authentication works best via [AppRouter](https://www.npmjs.com/package/@sap/approuter) (e.g. using a UAA configuration),
Authentication works best via [AppRouter](https://www.npmjs.com/package/@sap/approuter) (e.g. using a UAA configuration),
as the auth token is forwarded via authorization header bearer token by AppRouter to backend websocket call.

CDS auth strategy (e.g. `cds.auth.kind: 'xsuaa'`) is applied and CDS auth middleware processes the auth token and
Expand Down
2 changes: 1 addition & 1 deletion src/redis/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ const createClientBase = (options = {}) => {
tls: !!credentials.tls,
port: credentials.port,
...requiresRedis?.options?.socket,
...options?.config.socket,
...options?.config?.socket,
};
const redisOptions = {
...requiresRedis?.options,
Expand Down

0 comments on commit 4cb7667

Please sign in to comment.