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.

Sign up for GitHub

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

Jump to bottom

redis commander: handle none redis_user #809

Merged
merged 2 commits into from
Sep 17, 2024
Merged

redis commander: handle none redis_user #809

merged 2 commits into from
Sep 17, 2024

Conversation

YuryHrytsuk
Copy link
Collaborator

@YuryHrytsuk YuryHrytsuk commented Sep 17, 2024 •
edited
Loading

What do these changes do?

On some deployments we don't use REDIS_USER. This PR handles this case

FYI: @sanderegg

Related issue/s

Related PR/s

Checklist

  • I tested and it works

@YuryHrytsuk YuryHrytsuk added the t:enhancement New feature or request label Sep 17, 2024
@YuryHrytsuk YuryHrytsuk self-assigned this Sep 17, 2024
@YuryHrytsuk YuryHrytsuk merged commit b1e496d into ITISFoundation:main Sep 17, 2024
3 checks passed
@YuryHrytsuk YuryHrytsuk deleted the redis-commander-support-null-user branch September 17, 2024 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mrnicegyu11 mrnicegyu11 mrnicegyu11 approved these changes

Assignees

@YuryHrytsuk YuryHrytsuk

Labels
t:enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@YuryHrytsuk @mrnicegyu11