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

find a better way to store API keys #1

Open
mikeantonacci opened this issue Dec 13, 2015 · 5 comments
Open

find a better way to store API keys #1

mikeantonacci opened this issue Dec 13, 2015 · 5 comments

Comments

@mikeantonacci
Copy link
Collaborator

That Resources.resw file is really intended for storing strings for localization purposes, using it this way is kind of a hack. It's not clear what the correct/standard way to do this is...

@RitwikGupta
Copy link
Member

So we're supposed to obfuscate and encrypt the key to store it as a string in the app because it can easily be disassembled

@mikeantonacci
Copy link
Collaborator Author

Yeah, that's the other (and more important) problem, even when I grepped for the keys in the project folder to make sure I wasn't committing them I saw them in some binary files in the Debug directory...it's an ugly temporary solution so that I could get the keys out of the source code and get something on GitHub faster.

@RitwikGupta
Copy link
Member

Did you .gitignore those files?

@mikeantonacci
Copy link
Collaborator Author

Visual Studio generates a .gitignore by default that includes those build directories and a bunch of other IDE generated nonsense, so they were ignored already.

@epicstar epicstar added this to the First Release Milestone milestone May 7, 2016
@epicstar
Copy link
Member

epicstar commented May 7, 2016

May not find a better way so if we still see this open and we're ready to release, let's close this issue or move it to another milestone

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

3 participants