Fix `acra-keys list` with keystore v1 - Githubissues

cossacklabs / acra

Database security suite. Database proxy with field-level encryption, search through encrypted data, SQL injections prevention, intrusion detection, honeypots. Supports client-side and proxy-side ("transparent") encryption. SQL, NoSQL.

https://www.cossacklabs.com/acra/

Apache License 2.0

1.34k stars 128 forks source link

Fix `acra-keys list` with keystore v1 #530

Closed G1gg1L3s closed 2 years ago

G1gg1L3s commented 2 years ago

This PR fixes issues with acra-keys that were discovered at https://github.com/cossacklabs/product-docs/pull/245#discussion_r852193264.

The changes:

keystore.ListKeys now lists files only once, if public key path is the same as private key one.
in log key names secure is no longer interpreted as client id

Checklist

[x] Change is covered by automated tests
[x] The coding guidelines are followed
[x] Public API has proper documentation in the Acra documentation site or has PR on documentation repository with new changes
[x] ~CHANGELOG.md is updated (in case of notable or breaking changes)~
[x] CHANGELOG_DEV.md is updated
[x] ~Benchmark results are attached (if applicable)~
[x] ~Example projects and code samples are up-to-date (in case of API changes)~