Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐞 Redis 컨테이너 연결 실패 #463

Closed
kimyu0218 opened this issue Jan 12, 2024 · 0 comments · Fixed by #464
Closed

🐞 Redis 컨테이너 연결 실패 #463

kimyu0218 opened this issue Jan 12, 2024 · 0 comments · Fixed by #464
Assignees
Labels
Milestone

Comments

@kimyu0218
Copy link
Collaborator

kimyu0218 commented Jan 12, 2024

Description

redis를 도커 컨테이너로 띄우고, docker inspect로 redis 서버의 호스트를 집어넣었는데 localhost로 접속하는 문제가 발생했다.

image

How to reproduce it

CacheModule.registerAsync({
  inject: [ConfigService],
  useFactory: (configService: ConfigService): RedisClientOptions => {
    return {
      store: redisStore,
      host: configService.get('CACHE_HOST'),
      port: configService.get<number>('CACHE_PORT'),
     } as RedisClientOptions;
  },
}),

Todo

  • CacheConfigModule 전역으로 설정
  • useFactory 타입 오류 수정
    • 올바르지 않은 반환값이 들어가면서 localhost 기본값으로 접속

ETC

@kimyu0218 kimyu0218 added the bug label Jan 12, 2024
@kimyu0218 kimyu0218 self-assigned this Jan 12, 2024
@kimyu0218 kimyu0218 added the BE label Jan 12, 2024
@kimyu0218 kimyu0218 added this to the version 1.0.0 milestone Jan 12, 2024
@github-project-automation github-project-automation bot moved this to 📋 Backlog in web09-MagicConch Jan 12, 2024
@kimyu0218 kimyu0218 linked a pull request Jan 12, 2024 that will close this issue
kimyu0218 added a commit that referenced this issue Jan 12, 2024
@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in web09-MagicConch Jan 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

1 participant