Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Add Config File Redis Support #15343

Closed
wants to merge 1 commit into from
Closed

Conversation

berksmbl
Copy link

Added redis support for the migrate command.

@berksmbl berksmbl requested a review from a team as a code owner March 28, 2023 16:12
@reivilibre
Copy link
Contributor

Sorry, the migrate command was only intended for porting from old-style env vars, which we no longer recommend. It doesn't make sense to add more env vars to the template, since it's only there for backwards compatibility.

See #5518 (comment) for more on this.

As such, I will turn down this PR.

@reivilibre reivilibre closed this Mar 30, 2023
@berksmbl
Copy link
Author

I'm preparing a one-click installation template on EasyPanel for Synapse, and I want to add a Redis option, but currently there is no way to add the URL and Password information for Redis with ENV variables.

@berksmbl
Copy link
Author

berksmbl commented Apr 1, 2023

@reivilibre

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

Successfully merging this pull request may close these issues.

2 participants