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

revert redis pw change #214

Closed
wants to merge 1 commit into from
Closed

revert redis pw change #214

wants to merge 1 commit into from

Conversation

m-1-k-3
Copy link
Member

@m-1-k-3 m-1-k-3 commented Jan 27, 2022

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    Bug fix

  • What is the current behavior? (You can also link to an open issue here)
    In PR Reporting templates, cleanup #213 I changed the redis PW during installation to a random value. As the EMBA external directory is included in our docker container we can't change the password during installation. My fault :)

  • What is the new behavior (if this is a feature change)? If possible add a screenshot.
    Revert it to default password

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
    The installer should be able to handle it. If someone already has issues it is possible to change the password manually in redis.conf and configuration.ini

@m-1-k-3 m-1-k-3 added bug Something isn't working docker docker related things labels Jan 27, 2022
@m-1-k-3 m-1-k-3 requested a review from p4cx January 27, 2022 10:30
@m-1-k-3 m-1-k-3 marked this pull request as draft January 27, 2022 10:41
@m-1-k-3
Copy link
Member Author

m-1-k-3 commented Jan 31, 2022

I think it does not matter ... letz leave it for now. If someone runs into troubles we can take another look at it

@m-1-k-3 m-1-k-3 closed this Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working docker docker related things
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant