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

Failed to start redisProvider #1994

Closed
mzico opened this issue Jul 4, 2020 · 19 comments
Closed

Failed to start redisProvider #1994

mzico opened this issue Jul 4, 2020 · 19 comments
Assignees
Labels
Milestone

Comments

@mzico
Copy link
Contributor

mzico commented Jul 4, 2020

I am little confused if I should open this such issue under oxAuth or oxTrust. But, I can't setup external Redis Cache in 4.2.
Here is the testing situation: https://youtu.be/bTfRNttrf2k

@yuriyz
Copy link
Contributor

yuriyz commented Jul 4, 2020

@mzico would you please attach oxauth.log in TRACE log level?

@mzico
Copy link
Contributor Author

mzico commented Jul 4, 2020

@yuriyz : sure, will do it today.

@yuriyz
Copy link
Contributor

yuriyz commented Jul 4, 2020

@mzico great. Between, does it work if you set useSsl=false ?

@mzico
Copy link
Contributor Author

mzico commented Jul 4, 2020

@yuriyz : I think it's actually oxTrust issue. I have recorded a screencast which showing that only clear text password is being accepted by oxTrust and "Connection is successful".
After entering clear text ( first time ), oxTrust encode it and if I hit 'enter' again.. it's failing.

https://youtu.be/4wct6ZI147w

@yuriyz yuriyz transferred this issue from GluuFederation/oxAuth Jul 6, 2020
@yuriyz yuriyz added the bug label Jul 6, 2020
@yuriyz yuriyz added this to the 4.2 milestone Jul 6, 2020
@yuriyz yuriyz removed their assignment Jul 6, 2020
@yuriyz
Copy link
Contributor

yuriyz commented Jul 6, 2020

@mzico Got it, moved to oxTrust project and assigned to @syntrydy to check.

@syntrydy
Copy link
Collaborator

syntrydy commented Jul 6, 2020

@mzico how can i replicate this?

@mzico
Copy link
Contributor Author

mzico commented Jul 6, 2020

@syntrydy : my procedure is added in this comment. I have a redis server in cloud, if you want.. I can fire up that machine for your testing.

@syntrydy
Copy link
Collaborator

syntrydy commented Jul 6, 2020

@mzico In my case i was able to connect to redis. Please ive a try with the upcomming package. I changed the code to provide more information about the error.

syntrydy added a commit that referenced this issue Jul 8, 2020
syntrydy added a commit that referenced this issue Jul 8, 2020
@syntrydy
Copy link
Collaborator

syntrydy commented Jul 8, 2020

@mzico This is now fixed

@syntrydy syntrydy closed this as completed Jul 8, 2020
@mzico
Copy link
Contributor Author

mzico commented Jul 15, 2020

@syntrydy : Problem is still there in 4.2.0.

As discussed this is what happening: https://youtu.be/ylSGA9Ro3H8

  • Whenever we are clear text password for redis configuration, it's working.
  • After 'Update' / 'Save Configuration', oxTrust is encoding it.
  • Encoded value looks good.. BUT
  • oxTrust can't deal with encoded redis password and hence it's failing to complete configuration.

@mzico mzico reopened this Jul 15, 2020
@syntrydy
Copy link
Collaborator

@mzico i think you are not using that form as expected, to configure redis,
image
That is first step. In the video you don't select that.

@syntrydy
Copy link
Collaborator

image
The password display on the UI is not the encoded password.

@mzico
Copy link
Contributor Author

mzico commented Jul 21, 2020

@syntrydy @yuriyz : thanks for the fix in 4.2. Can we fix this in 3.1.8? As it's the last & long term version of v3, it will be great to have this fix here. Thoughts?

@mzico mzico reopened this Jul 21, 2020
@syntrydy
Copy link
Collaborator

@mzico do you have a video showing the behavior in 3.1.8?

@mzico
Copy link
Contributor Author

mzico commented Jul 22, 2020

@syntrydy : I will prepare one and share...

@syntrydy
Copy link
Collaborator

@mzico okey

@mzico
Copy link
Contributor Author

mzico commented Jul 27, 2020

@syntrydy : here is the video.

@syntrydy
Copy link
Collaborator

@mzico i think it is better to create new issue and tag it 3.1.8

@syntrydy syntrydy closed this as completed Aug 2, 2020
@mzico
Copy link
Contributor Author

mzico commented Aug 2, 2020

Done: #2008

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

No branches or pull requests

3 participants