Browse Source
Feature/add error handling for redis connections (#2179)
* Add error handling
* Update changelog
pull/2183/head
Thomas Kaul
2 years ago
committed by
GitHub
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with
30 additions and
4 deletions
-
CHANGELOG.md
-
apps/api/src/app/redis-cache/interfaces/redis-cache.interface.ts
-
apps/api/src/app/redis-cache/interfaces/redis-store.interface.ts
-
apps/api/src/app/redis-cache/redis-cache.service.ts
|
|
@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 |
|
|
|
|
|
|
|
## Unreleased |
|
|
|
|
|
|
|
### Added |
|
|
|
|
|
|
|
- Added error handling for the _Redis_ connections to keep the app running if the connection fails |
|
|
|
|
|
|
|
### Fixed |
|
|
|
|
|
|
|
- Fixed the missing values in the holdings table |
|
|
|
|
|
@ -0,0 +1,7 @@ |
|
|
|
import { Cache } from 'cache-manager'; |
|
|
|
|
|
|
|
import type { RedisStore } from './redis-store.interface'; |
|
|
|
|
|
|
|
export interface RedisCache extends Cache { |
|
|
|
store: RedisStore; |
|
|
|
} |
|
|
@ -0,0 +1,8 @@ |
|
|
|
import { Store } from 'cache-manager'; |
|
|
|
import Redis from 'redis'; |
|
|
|
|
|
|
|
export interface RedisStore extends Store { |
|
|
|
getClient: () => Redis.RedisClient; |
|
|
|
isCacheableValue: (value: any) => boolean; |
|
|
|
name: 'redis'; |
|
|
|
} |
|
|
@ -1,14 +1,21 @@ |
|
|
|
import { ConfigurationService } from '@ghostfolio/api/services/configuration/configuration.service'; |
|
|
|
import { UniqueAsset } from '@ghostfolio/common/interfaces'; |
|
|
|
import { CACHE_MANAGER, Inject, Injectable } from '@nestjs/common'; |
|
|
|
import { Cache } from 'cache-manager'; |
|
|
|
import { CACHE_MANAGER, Inject, Injectable, Logger } from '@nestjs/common'; |
|
|
|
|
|
|
|
import type { RedisCache } from './interfaces/redis-cache.interface'; |
|
|
|
|
|
|
|
@Injectable() |
|
|
|
export class RedisCacheService { |
|
|
|
public constructor( |
|
|
|
@Inject(CACHE_MANAGER) private readonly cache: Cache, |
|
|
|
@Inject(CACHE_MANAGER) private readonly cache: RedisCache, |
|
|
|
private readonly configurationService: ConfigurationService |
|
|
|
) {} |
|
|
|
) { |
|
|
|
const client = cache.store.getClient(); |
|
|
|
|
|
|
|
client.on('error', (error) => { |
|
|
|
Logger.error(error, 'RedisCacheService'); |
|
|
|
}); |
|
|
|
} |
|
|
|
|
|
|
|
public async get(key: string): Promise<string> { |
|
|
|
return await this.cache.get(key); |
|
|
|