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 SSL URI not reconized as a Redis type discoverer #1259

Closed
4 tasks done
Freezystem opened this issue Nov 17, 2023 · 0 comments · Fixed by #1260
Closed
4 tasks done

Redis SSL URI not reconized as a Redis type discoverer #1259

Freezystem opened this issue Nov 17, 2023 · 0 comments · Fixed by #1260

Comments

@Freezystem
Copy link
Contributor

Prerequisites

Please answer the following questions for yourself before submitting an issue.

  • I am running the latest version
  • I checked the documentation and found no answer
  • I checked to make sure that this issue has not already been filed
  • I'm reporting the issue to the correct repository

Current Behavior

Moleculer does not recognize redis SSL URIs as valid Redis URIs.

Expected Behavior

Redis SSL URI should be identified as a Redis type discoverer.

Failure Information

Issue can be mitigated using more explicit config:

discoverer: {
    type: "Redis",
    options: { redis: "rediss://redis-server:6379" },
},

Steps to Reproduce

Start a broket with rediss:// URI as discoverer

Reproduce code snippet

const broker = new ServiceBroker({
    discoverer: "rediss://redis-server:6379"
});

Context

  • Moleculer version: 0.14.32
  • NodeJS version: 20.8.0
  • Operating System: Ubuntu 23.10
  • Redis: 7.2.2

Failure Logs

Unable to create ServiceBroker. BrokerOptionsError: Invalid Discoverer type 'rediss://redis-server:6379'
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

Successfully merging a pull request may close this issue.

1 participant