refactor: remove cosmos/keyring

[rootulp]

Description

Resolves #9568 which should be re-opened and closed after this PR is merged
Resolves #5759
Likely resolves #10818

The replace for 99designs/keyring to a Cosmos fork of keyring (cosmos/keyring) was added in #9563

Removal of the replace was tracked in #9568 but never completed.

The original issue was fixed in upstream: 99designs/keyring#87 so this PR removes the replace and upgrades to the latest release of 99designs/keyring.

---

Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

• included the correct type prefix in the PR title
• added ! to the type prefix if API or client breaking change
• targeted the correct branch (see PR Targeting)
• provided a link to the relevant issue or specification
• followed the guidelines for building modules
• included the necessary unit and integration tests
• added a changelog entry to CHANGELOG.md
• included comments for documenting Go code
• updated the relevant documentation or specification
• reviewed "Files changed" and left comments if necessary
• confirmed all CI checks have passed

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.

I have...

• confirmed the correct type prefix in the PR title
• confirmed ! in the type prefix if API or client breaking change
• confirmed all author checklist items have been addressed
• reviewed state machine logic
• reviewed API design and naming
• reviewed documentation is accurate
• reviewed tests and test coverage
• manually tested (if applicable)

[rootulp]

I suspect that upgrading to 99designs/keyring v1.2.1 will also resolve #10818 because it includes 99designs/keyring#102

[rootulp]

I suspect this will also resolve #5759

[tac0turtle]

should do some testing before we merge. Thank you for opening this PR

@jleni any chance you can approve this

[codecov]

Codecov Report

Merging #12576 (9c9f25f) into main (da7f046) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main   #12576   +/-   ##
=======================================
  Coverage   65.20%   65.20%           
=======================================
  Files         694      694           
  Lines       71855    71855           
=======================================
+ Hits        46851    46852    +1     
+ Misses      22374    22372    -2     
- Partials     2630     2631    +1     

Impacted Files	Coverage Δ
x/group/keeper/keeper.go	58.66% <0.00%> (+0.39%)	⬆️

[jleni]

Should we test that this resolves?

• keyring custom path is not supported #10818
• Upgrade the usage of go-keychain #5759

or should we keep these issues open and review then later?

[tac0turtle]

Should we test that this resolves?

• keyring custom path is not supported #10818
• Upgrade the usage of go-keychain #5759

or should we keep these issues open and review then later?

ideally yes, would be nice to test

[rootulp]

Can confirm this resolves #5759 (updated PR description)

Before	After

[raynaudoe]

Hi! I can also confirm that no go-keychain related errors are thrown when building.
Tested on: macos 12.4, M1 arch.

[tac0turtle]

@Mergifyio backport release/v0.46.x

[mergify]

backport release/v0.46.x

✅ Backports have been created

• #12707 refactor: remove cosmos/keyring (backport #12576) has been created for branch release/v0.46.x