Comments (22)
I just want to add to this that getItem(key)
returns null
on Android and undefined
on iOS when no value exists at that key. Pretty minor issue, but just wanted to give a heads up to anyone looking at this.
from react-native-sensitive-info.
Thank you for letting us know. We're glad this is being helpful. Right now, we are pretty busy with other projects, but pull requests are always welcome!
from react-native-sensitive-info.
Hi guys 👋
Thanks for the report. There were no reason regressing this issue. In fact, it may have happend when I merged the keystore branch within the master branch, because the keystore branch had many differences in android's files and this fix wasn't included there.
So, we should only open a new PR for that using the master branch again. I may have do this later, however feel free doing that.
from react-native-sensitive-info.
The issue still exists in v6.0.0-alpha.9.
For now, I decided not to use getAllItems.
from react-native-sensitive-info.
@randycoulman I assume the iOS API should align with android's API in this case?
from react-native-sensitive-info.
I'd probably go the other way and make them both like the iOS API. That way, we'd be adding information to the Android API rather than removing information from the iOS API. Either way, it's going to be a breaking change somewhere.
from react-native-sensitive-info.
Thanks for building this library! Experiencing the same issue. If we're fixing the return values, shouldn't the iOS return value be one array rather than a nested array, i.e., return:
[
{ service: 'app', key: 'foo', value: 'bar' },
{ service: 'app', key: 'baz', value: 'quux' }
]
rather than
[
[
{ service: 'app', key: 'foo', value: 'bar' },
{ service: 'app', key: 'baz', value: 'quux' }
]
]
Happy to work on a PR for that if it's helpful. I don't have as much experience with Android React-Native so probably wouldn't be as helpful with reworking that section.
from react-native-sensitive-info.
@YPCrumble Yes, I agree with you. I forgot about the nested array in my earlier response.
So, my (non-authoritative) opinion is that we should return an array of objects, where the objects have the service
, key
, and value
keys in it.
from react-native-sensitive-info.
Hello guys, sorry for delaying the answer once again. I'll take a look in android in this week. About iOS, I'm afraid I wouldn't because I don't have macOS available.
from react-native-sensitive-info.
Moreover, the example in the README is a bit misleading. From looking at it, I expected values
to be an array, and have been cracking my head over this.
It would be nice to have an example output for each command, as well as return type, etc.
from react-native-sensitive-info.
Has this been resolved?
from react-native-sensitive-info.
opened a PR for this at #168
from react-native-sensitive-info.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
from react-native-sensitive-info.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
from react-native-sensitive-info.
I find the issue still exist because v6.0.0-alpha.5
doesn't contain #168. It's still use WritableMap
and same for master branch https://github.com/mCodex/react-native-sensitive-info/blob/master/android/src/main/java/dev/mcodex/RNSensitiveInfoModule.java#L241 .
Something wrong happened in release process?
from react-native-sensitive-info.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
from react-native-sensitive-info.
The issue still exists in v6.0.0-alpha.6.
from react-native-sensitive-info.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
from react-native-sensitive-info.
It looks like #214 stepped on this fix from #168
Is there a reason that this was regressed?
from react-native-sensitive-info.
I am still having this issue. Wasn't this fixed?
from react-native-sensitive-info.
const RNSIResponseNormal: SensitiveInfoEntry[] | [SensitiveInfoEntry[]] = APP_FLAGS.isIOS
? RNSIResponse[0]
: RNSIResponse
from react-native-sensitive-info.
Hey! its a simple hotFix,
but you can use the follow workaround to solve this problem :
export const getAllSecureItem = async () => {
try {
const allSecureItems = await SInfo.getAllItems({
sharedPreferencesName,
keychainService,
});
return Platform.OS === 'ios'
? allSecureItems[0]
: Object.entries(allSecureItems)?.map((secureItem) => {
const [key, value] = secureItem;
return { key: key, value: value };
});
} catch (e) {
// Error handler here.
}
};
This function return a simple Array of Secure Items:
[ { key: "", value: "" }, { key: "", value: "" } ]
from react-native-sensitive-info.
Related Issues (20)
- After uninstall and install app stored information remained (IOS) HOT 4
- [IOS] Protected data not available yet. Retry operation HOT 16
- Android: Error in secure save : [Error: Failed to obtain information about key] HOT 1
- Storage size HOT 3
- is ios Keychain sandboxed? HOT 1
- Adding `kSecAttrSynchronizable` options throws error. HOT 1
- Remember fingerprint for X minutes HOT 1
- Static Values for Cryptography Which Exposes Data to Easy Decryption HOT 3
- v6.0.0-alpha10: update latest alpha build on npm? HOT 1
- Simple solution to the inconsistent return of getAllItems() HOT 1
- doesn't work on react-native 0.70rc2 / Mac catalyst / os error 34018 HOT 1
- Security issues HOT 3
- Android: Attempt to invoke interface method on a null object reference (patch-package fix available) HOT 1
- The credentials are getting lost on android 10 on app uninstall or clearing the app data. HOT 1
- Is this thing on? HOT 1
- Release version v6 on NPM package HOT 1
- I am facing issue with this package in windows system on some system when key is not available and when i try to call getItem method it's not even throwing error it will result in white screen issue. HOT 1
- getAllItems works without encryption but getItem does not HOT 1
- Types for getAllItems is wrong HOT 1
- Got the DecryptionFailed error on Android with Touch ID
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from react-native-sensitive-info.