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

Incorrect memo display. #694

Closed
moshenskyDV opened this issue Sep 14, 2023 · 4 comments · Fixed by #775
Closed

Incorrect memo display. #694

moshenskyDV opened this issue Sep 14, 2023 · 4 comments · Fixed by #775
Assignees
Labels
bug Something isn't working

Comments

@moshenskyDV
Copy link

Description

Got some small visual bug on hashscan. Account memo specificly for phrase "adap" displayed as strange symbol. Worth to mention that it's only word that cause this bug for me. Any other word works well.
Test account with reproduced issue: https://hashscan.io/testnet/account/0.0.1709113
Also screenshot attached.
image

Steps to reproduce

  1. Create new account with .setAccountMemo('adap')
  2. Check visual of memo on Hashscan.

Additional context

No response

Hedera network

mainnet, testnet, previewnet

Version

latest

Operating system

None

@moshenskyDV moshenskyDV added the bug Something isn't working label Sep 14, 2023
@quiet-node
Copy link
Member

quiet-node commented Sep 15, 2023

Hello, @moshenskyDV! Thanks for raising the issue! Could you please provide me with some more HashScan links to other cases where the memo displays correctly as you've set it using .setAccountMemo()?

@moshenskyDV
Copy link
Author

Correct displaying:
https://hashscan.io/testnet/account/0.0.2008265
.setAccountMemo('anothermemo')

BTW: tried to hit around incorrect one - and looks like I'm able to another incorrects.
https://hashscan.io/testnet/account/0.0.2008284
.setAccountMemo('adao')
https://hashscan.io/testnet/account/0.0.2008313
.setAccountMemo('adat')

@moshenskyDV
Copy link
Author

@quiet-node I did a couple of more tests and found that incorrect memo appears only from 4 symbol word.
Here is other examples:
https://hashscan.io/testnet/account/0.0.2008403
.setAccountMemo('aca1')

https://hashscan.io/testnet/account/0.0.2008437
.setAccountMemo('bcca')

@quiet-node
Copy link
Member

Hello, @moshenskyDV. Thank you for providing the information! It's worth noting that phrases with a the length that's a multiple of 4 are indeed suitable for base64 encoding. This is why the system decodes the 'memo' with the assumption that it's in base64 format, resulting in the appearance of those unusual characters.

We'll look more into this and will provide the fix soon!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
3 participants