matterhorn-chat / matterhorn

A feature-rich Unix terminal client for the Mattermost chat system
BSD 3-Clause "New" or "Revised" License
1.05k stars 77 forks source link

Support command substitutions in config commands #821

Closed vintagentleman closed 9 months ago

vintagentleman commented 9 months ago

Hi, thanks for this awesome project!

The docs for otptokencmd suggest providing a command that returns a one-time password directly:

# OTP token command. Optional. If this is specified, the provided
# command will be executed to obtain OTP token required by matterhorn
# 2FA. Resulting 6 digit code will be inserted into the OTP prompt
# automatically.
#
# otptokencmd: secret-tool lookup matterhorn OTP

However, I don't see how this approach can work with TOTPs, which are regenerated all the time. An option instead would be to use secret-tool to store the secret key and generate TOTPs dynamically via oathtool:

otptokencmd: oathtool --totp=SHA512 -b $(secret-tool lookup matterhorn otp_secret_key)

This config doesn't seem to work though, which I assume is due to the fact that secret-tool lookup matterhorn otp_secret_key is interpreted as a literal string rather than executed as a subcommand.

$ matterhorn
oathtool: base32 decoding failed: Base32 string is invalid
Error loading config: Could not execute OTP token command: failed
jtdaugherty commented 9 months ago

I recommend writing a shell script that does what you need and then use that as your otptokencmd. Wouldn't that meet the need?

vintagentleman commented 9 months ago

That did meet my need, though only hardcoding the absolute path to the script worked for me.

Thank you!