Skip to content
This repository has been archived by the owner on Feb 12, 2025. It is now read-only.

Some special characters from payment request are displayed wrong in Eclair mobile #44

Open
dpad85 opened this issue Apr 13, 2018 · 5 comments

Comments

@dpad85
Copy link
Member

dpad85 commented Apr 13, 2018

Issue by btcontract
Tuesday Jan 23, 2018 at 15:51 GMT
Originally opened as ACINQ/eclair#398


Some encoded special characters, such as apostrophe, are not properly displayed in Eclair mobile, for example:

photo_2018-01-23_17-45-08

Similar payment request in another app:

photo_2018-01-23_17-45-11

@dpad85
Copy link
Member Author

dpad85 commented Apr 13, 2018

Comment by verretor
Tuesday Jan 23, 2018 at 16:36 GMT


It seems the Android app is escaping HTML which is good for security in my opinion although a bit ugly.

@dpad85
Copy link
Member Author

dpad85 commented Apr 13, 2018

Comment by btcontract
Tuesday Jan 23, 2018 at 16:49 GMT


@verretor is this relevant inside of Android app? A far as I know escaping is done to prevent javascript execution on websites.

@dpad85
Copy link
Member Author

dpad85 commented Apr 13, 2018

Comment by verretor
Tuesday Jan 23, 2018 at 17:16 GMT


Could be SQL injection. Eclair uses SQLite.

There are some characters that cause problems in some very specific situations. Imagine your app uses UTF-8 encoding and accepts 2 byte characters such as 뼢 but your database has an encoding that only accepts 1 byte characters.
뼢 is 0xbf22 (2 byte) but when translated into 1 byte, it becomes:
0xbf 0x22 = ¿"

Notice the double quote. That's just one of many examples and it requires a lot of tests to make sure it is really safe.

I think it's better to be too careful than not enough but that's just my opinion.

@dpad85
Copy link
Member Author

dpad85 commented Apr 13, 2018

Comment by dpad85
Tuesday Apr 03, 2018 at 16:36 GMT


@btcontract By chance did you keep the payment request with the Elaine's Idle Mind description ? I can't reproduce this issue, even with some exotic characters.

@dpad85
Copy link
Member Author

dpad85 commented Apr 13, 2018

Comment by btcontract
Tuesday Apr 03, 2018 at 17:16 GMT


no, sorry, it's been a long time.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant