Android RSA.decrypt also expects a base64 input

[pke]

This can't be right?

react-native-simple-crypto/android/src/main/java/com/pedrouid/crypto/RSA.java

Line 155 in c338e2e

byte[] cipherBytes = Base64.decode(message, Base64.DEFAULT);

The RSA.decrypt and RSA.decrypt64 both threat input as base64. Shouldn't the non base64 version just take the input string without decoding it first?

[ghbutton]

Yea that looks wrong

[ghbutton]

Actually I am not sure, will try to dig in a little more.

[ghbutton]

When we encrypt we store the intermediate as base64, which is why when we decrypt we assume the input is coming in as base64

[pke]

That's what a decrypt64 function would be for.
Another scenario is that this lib is used only for decryption of data encrypted somewhere else.
So a binary input would be preferable. Especially one that does not need conversion between JS/Native. Maybe using ArrayBuffer or UINT8? Couldn't find anything in the docu which datatype is simply mapped and has no transfer overhead.

[ghbutton]

I think using ArrayBuffers everywhere would make more sense, or we could try to differentiate the input based on if it is a string or not similar to the SHA methods.

[pke]

I think ArrayBuffer would need no translation between native an JS, if I am correct? (The RN module docs are not the best).

[ghbutton]

We have array buffers in other places, but we translate them to base64 in most cases. I dont think we can translate directly from javascript to native data types.

See: https://github.com/ghbutton/react-native-simple-crypto/blob/master/index.js#L76

[pke]

There is an undocumented WritableArray that we could use. I'll link it here later.

[ghbutton]

Will look at this one as well.