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

keychain + sharedpreferences support #12

Closed
geirman opened this issue Oct 10, 2015 · 18 comments
Closed

keychain + sharedpreferences support #12

geirman opened this issue Oct 10, 2015 · 18 comments

Comments

@geirman
Copy link

geirman commented Oct 10, 2015

wondering if you plan to also include SharedPreferences support for Android. A single component that abstracts both would be awesome.

@oblador
Copy link
Owner

oblador commented Oct 10, 2015

It's definitely planned, but no ETA right now. A PR would be highly appreciated!

On Sat, Oct 10, 2015 at 10:59 AM, Chris Geirman notifications@github.com
wrote:

wondering if you plan to also include SharedPreferences support for Android. A single component that abstracts both would be awesome.

Reply to this email directly or view it on GitHub:
#12

@chrisvariety
Copy link

I think Keystore would be more appropriate? http://developer.android.com/training/articles/keystore.html

SharedPreferences isn't encrypted.

@corymsmith
Copy link

Was having a look at this the other day, may do an implementation with https://facebook.github.io/conceal/ and ship a PR over.

@geirman
Copy link
Author

geirman commented Jan 8, 2016

That looks pretty awesome, @corymsmith

@corymsmith
Copy link

@geirman I'll probably start digging into this today :)

@oblador
Copy link
Owner

oblador commented Jan 8, 2016

@corymsmith 👏 🙇

@corymsmith
Copy link

Ran into some conflicts with React Native so had to update Conceal to fix it.

facebookarchive/conceal#105

Made some great progress today! It looks like there are some issues with Conceal on recent Android versions that are in progress so will have to see how that progresses.!

@liorsaar1
Copy link

@corymsmith are you creating an android version for this ? would like to collaborate - liorsaar1@gmail.com

@corymsmith
Copy link

@liorsaar1 Yes, I have some updates in the android-support branch of my fork. I was having some issues unencrypting and still working through the best approach for Android. I broke my arm so can't do much until thats all sorted unfortunately

@oblador
Copy link
Owner

oblador commented Feb 12, 2016

@corymsmith: 👏 so excited about this! Get well 😷

@paulondc
Copy link

Any updates about the android support ? I'm very excited about it :)

@corymsmith
Copy link

@paulondc I've been super busy as of late with a few other open source projects and my new consulting company so haven't looked at this in a it but it was super close when I last left off. I'll be needing this in a few apps in the near future so hoping to pick it back up and wrap a release in the next few weeks or less

@paulondc
Copy link

paulondc commented Apr 24, 2016

Thanks @corymsmith for all effort you are putting on this 👏

I wonder if we could help you somehow, since this is going to introduce the initial support for android, I think we could have a special branch under this repository for this feature, so everybody interested on this could try it, watch the progress, create pull requests, etc..

What do you guys think about it ?

@corymsmith
Copy link

@vonovak
Copy link
Collaborator

vonovak commented May 9, 2016

nice work! @paulondc when can we expect the android support in npm? Thanks!

@vonovak vonovak mentioned this issue May 15, 2016
@oblador
Copy link
Owner

oblador commented May 31, 2016

Finally 0.3.0 with android support is on npm. Thanks to everyone involved making this happen!

@oblador oblador closed this as completed May 31, 2016
@rops
Copy link

rops commented Jun 20, 2016

Hi folks, I was just wondering if there was any particular (security or performance) reason why Android's KeyStore wasn't used

@vonovak
Copy link
Collaborator

vonovak commented Jun 20, 2016

Hello,
Someone started the android implementation a while ago and used the
Facebook conceal in it. It was, however, incomplete and wasn't merged. I
picked it up and finished it. Not sure what the initial reasons were but
IMHO there is no big difference. With conceal you get support in <= 4.3 and
perhaps greater flexibility in what you can save.
On Jun 20, 2016 19:07, "Daniele Rossetti" notifications@github.com wrote:

Hi folks, I was just wondering if there was any particular (security or
performance) reason why Android's KeyStore wasn't used


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#12 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABfmw3qMh_PueXk-s3eEikI9XvBtMFWHks5qNsizgaJpZM4GMlNV
.

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

8 participants