fix: skip Redis auth when credentials are unset with unit test (#541)

[Justxd22]

Description

• fix Redis cache config to skip AUTH when REDIS_PASSWORD is unset
• use redis://host:port when no credentials are provided
• keep REDIS_URI precedence when present
• add unit test coverage in test/unit/cache/client.spec.ts for:
  • no REDIS_URI + no password
  • password-based config
  • REDIS_URI precedence

Related Issue

#541

How Has This Been Tested?

npm run test:unit

Types of changes

• Bug fix (non-breaking change which fixes an issue)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my code changes.
• I added a changeset, or this is docs-only and I added an empty changeset.
• All new and existing tests passed.

[Copilot]

Pull request overview

This PR fixes Redis cache client configuration so the app does not attempt Redis AUTH when REDIS_PASSWORD is unset (and still respects REDIS_URI when provided), and adds unit tests plus a changeset to cover the expected configuration behavior.

Changes:

• Update getCacheConfig() to build an unauthenticated Redis URL when credentials are not provided, while keeping REDIS_URI precedence.
• Add unit tests covering unauthenticated config, authenticated config, and REDIS_URI precedence.
• Add a patch changeset documenting the fix.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
src/cache/client.ts	Adjusts Redis client options generation to avoid AUTH when password is unset.
test/unit/cache/client.spec.ts	Adds unit coverage for the updated Redis cache config behavior.
.changeset/thirty-turtles-design.md	Declares a patch release for the Redis config fix.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[coveralls]

coverage: 64.041% (+0.03%) from 64.015% — Justxd22:fix/541-redis-auth-unset-credentials into cameri:main

[cameri]

Is default in the Redis documentation? how does this default user come to be?

[Justxd22]

yes by default it's 'default'

https://redis.io/docs/latest/operate/rs/security/access-control/manage-users/default-user/
https://redis.io/docs/latest/operate/oss_and_stack/management/security/acl

[changeset-bot]

🦋 Changeset detected

Latest commit: f5f70ca

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
nostream	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR