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

Security Concern #2

Closed
paschaldev opened this issue Jun 26, 2020 · 2 comments
Closed

Security Concern #2

paschaldev opened this issue Jun 26, 2020 · 2 comments

Comments

@paschaldev
Copy link

Is it safe to commit your keystore file as well as the signing credentials in a repo?

@FanchenBao
Copy link

The .keystore file obviously must be git ignored.

The password for keystore and key shall not be committed as well. The official doc of react native provides another option to store them in ~/.gradle/gradle.properties, which is not part of the app. This way, there is no chance of leaking secure information.

However, I presume that if we are using the global ~/.gradle/gradle.properties, each keystore and alias must identify the app itself to prevent conflict with future projects.

@osamaqarem
Copy link
Owner

Is it safe to commit your keystore file as well as the signing credentials in a repo?

If it's a private repo and only people who are authorized to deploy the app can access it, I suppose its fine to commit the key and credentials. Otherwise it should be kept on your machine.

For signing credentials, I will add to the reply above that you can also use local.properties (or create any *.properties file) instead of gradle.properties to store them and not commit it to repo. But you will need to add more code to app/build.gradle to read the credentials from that file.

Because there are many approaches to securing your key/credentials, I think its better if we keep the instructions for the guide simple for now (add a note with a link to this issue maybe). People who are concerned about security will probably look into it and figure something out that works for them.

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