Skip to content

redis_url should be secret #1300

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

Merged
merged 1 commit into from
Aug 1, 2023

Conversation

kajinamit
Copy link
Contributor

The redis_url option can contain password string in case authentication is enabled in Redis, thus its value should be hidden in log output.

@tobias-urdin
Copy link
Contributor

@Mergifyio rebase

The redis_url option can contain password string in case authentication
is enabled in Redis, thus its value should be hidden in log output.
@mergify
Copy link
Contributor

mergify bot commented Aug 1, 2023

rebase

✅ Branch has been successfully rebased

@tobias-urdin tobias-urdin merged commit f0c0fe7 into gnocchixyz:master Aug 1, 2023
@tobias-urdin
Copy link
Contributor

@Mergifyio backport-to-4.5

@mergify
Copy link
Contributor

mergify bot commented Aug 1, 2023

backport -to-4.5

❌ No backport have been created

  • Backport to branch -to-4.5 failed

GitHub error: Branch not found

@tobias-urdin
Copy link
Contributor

@Mergifyio backport stable/4.5

@mergify
Copy link
Contributor

mergify bot commented Aug 1, 2023

backport stable/4.5

✅ Backports have been created

@kajinamit kajinamit deleted the redis_url-secret branch August 1, 2023 12:09
mergify bot added a commit that referenced this pull request Aug 1, 2023
@tobias-urdin
Copy link
Contributor

@Mergifyio backport stable/4.4

@mergify
Copy link
Contributor

mergify bot commented Aug 1, 2023

backport stable/4.4

✅ Backports have been created

tobias-urdin added a commit that referenced this pull request Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants