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

Erro ao conectar em Redis Host diferente de 127.0.0.1 #4

Open
bruno-bert opened this issue Apr 16, 2020 · 5 comments
Open

Erro ao conectar em Redis Host diferente de 127.0.0.1 #4

bruno-bert opened this issue Apr 16, 2020 · 5 comments

Comments

@bruno-bert
Copy link

O redisConfig não consegue entender o host e a porta configuradas, da forma que está passando no construtor. No seu código funcionou, pois o redis estava no docker e mapeou pro 127.0.0.1, e então, ele entendeu o host padrão. Peguei esse erro quando tentei fazer um container ler o redis de outro container, e neste caso, meu host do redis se chamava "redis". Pra arrumar, deve passar a url da conexão, dessa forma:

de:
const mailQueue = new Queue(RegistrationMail.key, redisConfig)

para:
const mailQueue = new Queue(RegistrationMail.key, redis://${redisConfig.host}:${redisConfig.port})

Espero poder ajduar alguem com essa dica! Valew!

@gpedro34
Copy link

@bruno-bert apenas vi depois de resolver o problema, mas a minha solução é diferente e usa o mesmo formato do código original.

redisConfig (src/config/redis.js) deveria ser:

export default {
  redis: {
    host: process.env.REDIS_HOST,
    port: Number(process.env.REDIS_PORT),
  },
}

em vez de:

export default {
    host: process.env.REDIS_HOST,
    port: process.env.REDIS_PORT,
}

@bruno-bert
Copy link
Author

Hum..estranho, pois o problema era no host e nao na porta. Seu codigo ainda esta rodando apontando pro localhost? Se sim, Testa ele rodando apontando pra outro host.
Ate mais!

@gpedro34
Copy link

@bruno-bert O Number(...) não é o que resolve o problema...
O problema é a estrutura do objecto.
O Bull espera um objecto com outro objecto (redis) dentro dele. Esse objecto é que deve conter as credenciais, não o objecto base.

@bruno-bert
Copy link
Author

Got it!! Entendi.

Sua solucao melhor que a minha. Faz um pull request ai!

gpedro34 added a commit to gpedro34/masterclass-nodejs-background-jobs that referenced this issue Nov 11, 2020
@gpedro34
Copy link

PR feita ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants