Cleanup keys.go in 24-host

[colin-axner]

Summary

During audit we noted the readability of keys.go could be improved. There were some unused functions and some constants in the file were not using a constant variable. We should always use consts and vars instead of hardcoded strings.

Key and Path functions are used for string vs byte representation. We should try to remove one of the functions if minimally used in the codebase to improve readability.

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged
• Contributor assigned/self-assigned

[colin-axner]

Ideally this comment is addressed too.

I think the functions could be renamed from KeyClientState to ClientStateKey etc