[Code]: Refactor the key name in the Keychain
iron-tech-space opened this issue · 3 comments
iron-tech-space commented
mikekazakov commented
Hi, this looks like a fully-fledged feature.
I agree that distinguishing password's location in the keychain by the port could be useful, and would be ok with such feature, given:
- the same logic applies to other connection types as well;
- default ports are handled too;
- there's an automatic migration procedure that can ensure that no passwords are lost after updating to a version with such distinction;
- all of that is covered with unit tests.
Would you like to work on that?
iron-tech-space commented
Oh, I'm ready to try, but I'm afraid I may not have enough skills in C++ and Objective-C++
mikekazakov commented
Well, if you want this feature implemented, go for it!
Someone needs to take the initiative, and I won’t have the bandwidth to do so in the foreseeable future.
Feel free to ask any code-related questions in the Development section of Discussions.