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

🚀 feat: Support Redis Clusters, Trusted Proxy Setting, And Toggle Meilisearch Indexing #5963

Merged
merged 9 commits into from
Feb 20, 2025

Conversation

danny-avila
Copy link
Owner

@danny-avila danny-avila commented Feb 20, 2025

Summary

Closes #5957

  • Allow disabling Meilisearch <> Database Indexing Sync by Instance
  • Added Redis cluster support with URL mapping, key prefix, max listeners, and optional TLS configuration using environment variables.
  • Allow Configuration of Trusted Proxy Express setting
  • Updated .env.example to include settings for Redis, TRUST_PROXY, and MeiliSearch indexing, ensuring the Express server uses the configurable proxy value.

Documentation: LibreChat-AI/librechat.ai#243

Other Changes

  • Removed the deprecated GitHub Question issue template to simplify repository maintenance.

Change Type

  • New feature (non-breaking change which adds functionality)
  • Documentation update

Testing

  • Verified Redis connectivity in both single-node and cluster modes.
  • Confirmed the TRUST_PROXY configuration by simulating reverse proxy requests to the Express server.
  • Ran the MeiliSearch index sync process to ensure appropriate error handling and proper indexing behavior.
  • Ensured that the removal of the GitHub template does not impact repository operations.

Checklist

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my code
  • I have commented in any complex areas of my code
  • I have made pertinent documentation changes
  • My changes do not introduce new warnings
  • I have written tests demonstrating that my changes are effective or that my feature works
  • Local unit tests pass with my changes
  • Any changes dependent on mine have been merged and published in downstream modules

@danny-avila danny-avila merged commit 1e625f7 into main Feb 20, 2025
5 checks passed
@danny-avila danny-avila deleted the feat/minor-updates branch February 20, 2025 22:39
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 this pull request may close these issues.

4 participants