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

RANGER-4865 : Add allowPublicKeyRetrieval option #341

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

kimjye-dev
Copy link

@kimjye-dev kimjye-dev commented Jul 18, 2024

What changes were proposed in this pull request?

https://issues.apache.org/jira/browse/RANGER-4865

(Please fill in changes proposed in this fix. Create an issue in ASF JIRA before opening a pull request and
set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g. RANGER-XXXX: Fix a typo in YYY))

How was this patch tested?

After applying what I wrote,
After ranger-admin setup, run ranger-admin start.

(Please explain how this patch was tested. Ex: unit tests, manual tests)
(If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)

  • Ranger : 2.4.0
  • Mysql : 8.0.36

# For mysql 8.0 and later versions
db_allow_public_key_retrieval=false
db_allow_public_key_retrieval
@kimjye-dev
Copy link
Author

Requesting review for this PR.
@kumaab @bhavikpatel9977 @mneethiraj

@bhavikpatel9977
Copy link
Contributor

@kimjye-dev Change is also required for dba_script.py file as well, Could you also update the change for ranger kms as well?

add : db_allow_public_key_retrieval
and fix : useSSL is false
add db_allow_public_key_retrieval in kms install.properties
@kimjye-dev
Copy link
Author

@bhavikpatel9977 After changing the dba_script.py file, ranger-admin ran normally.
I also changed ranger-kms, but I haven't tested this.
Please check again.

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

Successfully merging this pull request may close these issues.

2 participants