Follow-up from "Draft: Add key listing functionality, replace placeholder subcredentials with values from keystore"
The following discussion from !1660 (closed) should be addressed:
-
@nickm started a discussion: (+1 comment) I suggest that instead of stripping the extension unconditionally, we might want to make sure that it is a recognized extension. That way if we wind up with extra stuff in the keystore (like, file formats we don't recognize, or temporary files), we don't risk returning it and confusing ourselves.