Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EchoNest API Key Redaction #1907

Closed
nomadturk opened this issue Mar 11, 2016 · 2 comments
Closed

EchoNest API Key Redaction #1907

nomadturk opened this issue Mar 11, 2016 · 2 comments

Comments

@nomadturk
Copy link

Problem

When you run beet config command, all the api keys except echonest's are redacted.You can see it in cleartext.

Setup

  • OS: Debian
  • Python version: 2.7.9
  • beets version: 1.3.17
@nomadturk
Copy link
Author

Also as a side note, the beet config output also displays musicbrains username and password in cleartext as well. I think they should also be redacted for careless copy/pasters.

@sampsyo
Copy link
Member

sampsyo commented Mar 11, 2016

Thanks! I looked into this a little more closely, and I found that my key was getting redacted. The problem, it seems, is that it only gets redacted when the plugin is enabled—because plugins are how beets knows which fields are sensitive.

That's clearly not great, because it means it's possible to accidentally leak your key if you disable a plugin, but at least that explains why it's happening!

Anyway, the MB password thing was indeed just missing altogether; I added the redaction flag for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants