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

Allow other password authenticator to work #100

Closed
weideng1 opened this issue Feb 7, 2023 · 2 comments · Fixed by #101
Closed

Allow other password authenticator to work #100

weideng1 opened this issue Feb 7, 2023 · 2 comments · Fixed by #101

Comments

@weideng1
Copy link
Collaborator

weideng1 commented Feb 7, 2023

The current code only accepts com.datastax.bdp.cassandra.auth.DseAuthenticator and org.apache.cassandra.auth.PasswordAuthenticator, which unnecessarily limits the kind of CQL data stores that can be used as Origin or Target. The proposed change takes the same approach that CQL python driver takes, i.e. if it's not a DSEAuthenticator, it will assume the CQL data store uses a compatible password authenticator. When we need to handle more non-password authenticators, this segment will likely change back to switch statement, but that's going to be a bigger change any way.

┆Issue is synchronized with this Jira Task by Unito
┆Components: Proxy
┆Issue Number: ZDM-530

@ims-swilkinson
Copy link

We've hit this problem trying to migrate from DSE to Astra. For Astra it needs to support the "org.apache.cassandra.auth.AstraAuthenticator" authenticator. This seems to be compatible with "org.apache.cassandra.auth.PasswordAuthenticator" which is already supported.

@joao-r-reis joao-r-reis linked a pull request Nov 30, 2023 that will close this issue
@joao-r-reis
Copy link
Collaborator

We've hit this problem trying to migrate from DSE to Astra. For Astra it needs to support the "org.apache.cassandra.auth.AstraAuthenticator" authenticator. This seems to be compatible with "org.apache.cassandra.auth.PasswordAuthenticator" which is already supported.

the PR hasn't been merged yet but the fix is available in docker image datastax/zdm-proxy:sha-33db4fd if needed

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 a pull request may close this issue.

3 participants