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

Add Java implementation #2

Merged
merged 3 commits into from
May 17, 2018
Merged

Conversation

juliardi
Copy link
Contributor

@juliardi juliardi commented Nov 7, 2017

Hi,
Thank you for sharing the algorithm of Navicat password encryption and decryption also the python code implementation.
I'm happy to be able to port it to Java and if you would like please merge it to your repository.
Thank you,

Juliardi

@juliardi juliardi mentioned this pull request Nov 29, 2017
@juliardi
Copy link
Contributor Author

Hi,
It has been 2 months since I sent this pull request. Will you merge this or are there something wrong with the pull request?

@DoubleLabyrinth
Copy link
Contributor

DoubleLabyrinth commented Jan 27, 2018

Well, I refactor my code and update README.md just now. You can see it in /python3/NavicatCrypto.py.
Your code has nothing wrong. But I suggest you add Java implementation for Navicat12 password encryption/decryption in NCX (Navicat Connection eXport) file. I've implemented it in Python3 and you can take a reference.

Navicat12 changed the encryption algorithm for password in NCX file. If you have Navicat12, and export one of your connection configurations with password, you will find that encrypted password cannot be decrypted by previous algorithm I said.
For example, if you password is 12345678, connections.ncx generated by Navicat12 says encrypted password is 75008D0AE102C19EE3767E201AC9E4D2 while previous algorithm says it is 9AA92D8A5BDCB3F7.

@juliardi
Copy link
Contributor Author

Thank you for your information. I will try to port your new implementation to Java.

@DoubleLabyrinth DoubleLabyrinth merged commit 1594bd8 into HyperSine:master May 17, 2018
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