-
Notifications
You must be signed in to change notification settings - Fork 60
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
New redis configuration ssl parameters support #1018
Comments
Once implemented and 3.1.4 packages go out for testing, I'll check functionality. |
@yuriyz Please check it and let me know if you need anything more in this issue? |
looks good, one small improvement, when useSsl=false, set sslTrustStoreFilePath field disabled. |
@shekhar16 I think we should change |
#1018 : New redis configuration ssl parameters support
@yuriyz we had implemented it with jsoneditor.js which doesn't supporting hide/show on change functionality. |
got it, it's not critical anyway. |
The string is unchanged in RC4, please follow @afroDC's idea |
@sahiliamsso is right and that field, Also the caption under |
@yuriyz if I m going to change the label useSsl to useSSL than we need to change in oxauth code aswel. @sahiliamsso @afroDC @natt-tester Once @yuriyz confirms the changes than only we can change the label otherwise it will break the functionality. I updated the caption. here is screenshot. |
GluuFederation/oxTrust#1018 (cherry picked from commit ccdcbd5)
@shekhar16 changed in oxCore 3.1.4 and 4.0.0 (thus in oxauth too). |
changed |
Now on oxcore we have two more properties for SSL support:
See class here https://github.com/GluuFederation/oxCore/blob/5437b4200076eef81f43fa19e63989f1b8acbd43/oxService/src/main/java/org/xdi/service/cache/RedisConfiguration.java#L11
We need to support them on UI.
The text was updated successfully, but these errors were encountered: