tindroid icon indicating copy to clipboard operation
tindroid copied to clipboard

Avatars do not work

Open piratov opened this issue 1 year ago • 19 comments

If you are not reporting a bug, please post to https://groups.google.com/d/forum/tinode instead. https://groups.google.com/g/tinode/c/MIK7B0rrce0

Describe the bug You cannot set an avatar and the ones already installed from the web client are not displayed. Not set per user, group, or channel.

To Reproduce Use the possibilities that are available by installing an avatar in the android client.

Expected behavior Set avatar for user, group and channel from camera or gallery mobile application.

Screenshots ANDROID WEB

Tindroid (please complete the following information):

  • Version [0.19 and 0.20 (https://github.com/tinode/tindroid/tree/next)]
  • Device [X30 Cubot]
  • Android API Version -

Server

  • Type [api.tinode.co or your own setup]
  • Server type [unmodified or customized]
  • Server version [0.19, 0.20]

Additional context bugreport-X30_EEA-QP1A.190711.020-2022-08-25-15-44-19.zip

Console server Opening tinode /#contacts with avatars.(android) I2022/08/25 14:52:37 ws: session started _4KIaqdULl8 IP_ADDRESS:9441 2 I2022/08/25 14:52:37 in: '{"hi":{"dev":"clC3FqjmRIKRZkstmUxuIb:APA91bFMenksWZgfQ-mTjCuLOWqPveWnMCOKd6kkxOP-15xdD0xCSzQ41ZxO3i2yVp5lMy6vdABhIPfULEp8uNpdZlR9MMtYG5nMMJiYxiCjME3odGbIzuZjPN9U3gejkRKZ-WCv6q5g","id":"104062","lang":"ru_RU","ua":"Tindroid/20n (Android 10; ru_RU); tindroid/20n","ver":"0.20"}}' sid='_4KIaqdULl8' uid='' I2022/08/25 14:52:37 in: '{"login":{"id":"104063","scheme":"token","secret":"2WYMJdwbHit41xljFAABAAEA8f228kenCSc5SwDLfEv+8UX71Np49Q3sapZQapx7Ftw="}}' sid='_4KIaqdULl8' uid='' I2022/08/25 14:52:37 in: '{"sub":{"get":{"desc":{"ims":"2022-08-23T05:31:04.375Z"},"sub":{"ims":"2022-08-24T05:04:17.955Z"},"what":"desc sub tags cred","empty":false},"id":"104064","topic":"me"}}' sid='_4KIaqdULl8' uid='2WYMJdwbHis' I2022/08/25 14:52:37 in: '{"sub":{"get":{"desc":{"ims":"2022-08-23T05:31:04.375Z"},"sub":{"ims":"2022-08-24T05:04:17.955Z"},"what":"desc sub tags cred","empty":false},"id":"104065","topic":"me"}}' sid='_4KIaqdULl8' uid='2WYMJdwbHis' I2022/08/25 14:52:37 in: '{"sub":{"id":"104066","topic":"fnd"}}' sid='_4KIaqdULl8' uid='2WYMJdwbHis' I2022/08/25 14:52:37 in: '{"get":{"id":"104067","sub":{"ims":"2022-08-25T11:52:25.005Z"},"topic":"fnd","what":"sub"}}' sid='_4KIaqdULl8' uid='2WYMJdwbHis'

As a comparison, open tinode /#contacts WEB client, where there are avatars.(web) I2022/08/25 14:48:42 ws: session started kkpdTMKYC24 IP_ADDRESS:55275 1 I2022/08/25 14:48:42 in: '{"hi":{"id":"100998","ver":"0.20.0-rc1","ua":"TinodeWeb/0.20.0-rc1 (Chrome/97.0; Win32); tinodejs/0.20.0-rc1","lang":"en-US","platf":"web"}}' sid='kkpdTMKYC24' uid='' I2022/08/25 14:48:42 in: '{"login":{"id":"100999","scheme":"token","secret":"2WYMJdwbHivf1RljFAABAAEA0O4v2Oqi5/AubsJoTMMcE7mtbYUenJiXJknw6zzGsog="}}' sid='kkpdTMKYC24' uid='' I2022/08/25 14:48:42 in: '{"sub":{"id":"101000","topic":"me","get":{"what":"sub desc tags cred"}}}' sid='kkpdTMKYC24' uid='2WYMJdwbHis' I2022/08/25 14:48:42 media serve: OK, uid= 2WYMJdwbHis I2022/08/25 14:48:42 media serve: OK, uid= 2WYMJdwbHis

Attempting to set a gallery image or photo from the camera to an avatar.(android) I2022/08/25 14:40:18 ws: session started WS2bnUp9BI4 IP_ADDRESS:9372 1 I2022/08/25 14:40:18 in: '{"hi":{"dev":"clC3FqjmRIKRZkstmUxuIb:APA91bFZX-_kS_bY_2YxkLtkziQn7Jxu3k6clg2vAy_1kfqhIqNUuxWkfzmdFR8DVFI7QDSEKmnhjHLpEgugw0Ljw0qyXlTg2z5fEmZ_fGsyV6XAPMF7Q6LwKnxgbkmm7m4cUq2BXsYK","id":"130423","lang":"ru_RU","ua":"Tindroid/20n (Android 10; ru_RU); tindroid/20n","ver":"0.20"}}' sid='WS2bnUp9BI4' uid='' I2022/08/25 14:40:18 in: '{"login":{"id":"130424","scheme":"token","secret":"cxSpTP2DymeQ1BljFAABAAEAmaNwDr6Rpl0aKGVLmqgGKWE/IAao0owvQYqVKMkZ6Ts="}}' sid='WS2bnUp9BI4' uid='' I2022/08/25 14:40:18 in: '{"sub":{"get":{"desc":{"ims":"2022-08-22T08:49:50.906Z"},"sub":{"ims":"2022-08-24T04:49:21.003Z"},"what":"desc sub tags cred","empty":false},"id":"130425","topic":"me"}}' sid='WS2bnUp9BI4' uid='cxSpTP2Dymc' I2022/08/25 14:40:18 in: '{"sub":{"get":{"desc":{"ims":"2022-08-22T08:49:50.906Z"},"sub":{"ims":"2022-08-24T04:49:21.003Z"},"what":"desc sub tags cred","empty":false},"id":"130426","topic":"me"}}' sid='WS2bnUp9BI4' uid='cxSpTP2Dymc' I2022/08/25 14:40:18 in: '{"sub":{"id":"130427","topic":"fnd"}}' sid='WS2bnUp9BI4' uid='cxSpTP2Dymc' I2022/08/25 14:40:18 in: '{"get":{"id":"130428","sub":{"ims":"2022-08-25T11:40:00.404Z"},"topic":"fnd","what":"sub"}}' sid='WS2bnUp9BI4' uid='cxSpTP2Dymc'

A large image closes the application.(android) Sometimes when trying to add or save an avatar, there is a message “Failed” (action_failed)

piratov avatar Aug 25 '22 13:08 piratov

Do I understand correctly that you are using your own server?

or-else avatar Aug 25 '22 18:08 or-else

A large image closes the application.(android)

I don't see a crash report in Crashlytics. Is this your own build of Android?

or-else avatar Aug 25 '22 18:08 or-else

Please attach the original file you use as avatar. First zip it then attach to be sure it's not scaled down by github.

or-else avatar Aug 25 '22 18:08 or-else

Do I understand correctly that you are using your own server?

Yes, but the problem is common. It doesn't matter your own or downloaded github.

A large image closes the application.(android)

I don't see a crash report in Crashlytics. Is this your own build of Android?

I can repeat crash report on the downloaded github?

Please attach the original file you use as avatar. First zip it then attach to be sure it's not scaled down by github.

If you are talking about the problem of crash due to a larger image? I think any image is larger than allowed on the server.

Additional Information.. It seems I'm starting to understand what the collision with the avatar is. If you change the avatar in android, then the avatar in the web client disappears. And vice versa, if you set an avatar in the web client, then the avatar in the android client disappears.

piratov avatar Aug 26 '22 06:08 piratov

To create a bugreport, I used the latest available versions https://github.com/tinode/chat/tree/next https://github.com/tinode/tindroid/tree/next

bugreport-X30_EEA-QP1A.190711.020-2022-08-27-12-09-55.zip

Avatar I used Screenshot_20200502-193517.zip

piratov avatar Aug 27 '22 09:08 piratov

I can repeat crash report on the downloaded github?

Maybe. Did you try? I don't need the crash itself, I need the step to reproduce it. I know the crash happens when you scale the image too much, I just can't reproduce it.

or-else avatar Aug 27 '22 17:08 or-else

I can repeat crash report on the downloaded github?

Maybe. Did you try? I don't need the crash itself, I need the step to reproduce it. I know the crash happens when you scale the image too much, I just can't reproduce it.

here is an example photo from the camera

IMG_20220728_195342.zip or https://files.fm/u/zercugsfe

piratov avatar Aug 27 '22 19:08 piratov

Can you describe the steps to the crash?

or-else avatar Aug 27 '22 20:08 or-else

Never mind. I managed to reproduce the crash.

or-else avatar Aug 27 '22 22:08 or-else

Two videos one crash another avatar installation https://files.fm/u/4tvu68m4w

piratov avatar Aug 27 '22 23:08 piratov

Crash fixed in https://github.com/tinode/tindroid/commit/f29f8412d0861622e08feba08be71dd89dead32c Thanks.

or-else avatar Aug 28 '22 01:08 or-else

The broken avatars were due to a bug in webapp. I believe it's fixed now. Please double check.

or-else avatar Sep 01 '22 19:09 or-else

Unfortunately no change.

Additional difficulties have been added. It is not immediately possible to connect (503) to the server. And also log in the first time (409). When adding an avatar, the account is logged out.

https://files.fm/f/x845engkv video

piratov avatar Sep 01 '22 21:09 piratov

Did you update the server and both apps?

or-else avatar Sep 01 '22 22:09 or-else

Did you update the server?

Yes, devel last

piratov avatar Sep 01 '22 22:09 piratov

Anyway, I cannot reproduce. Please show the logcat.

or-else avatar Sep 01 '22 22:09 or-else

https://files.fm/f/c7hwf5x7f or logcat.txt

piratov avatar Sep 01 '22 23:09 piratov

You are using an android client prior to https://github.com/tinode/tindroid/commit/f29f8412d0861622e08feba08be71dd89dead32c Please use an updated client.

or-else avatar Sep 01 '22 23:09 or-else

You are using an android client prior to f29f841 Please use an updated client.

Yes, using last e8a1294 No more this error f29f841

app-release.apk.zip

piratov avatar Sep 02 '22 00:09 piratov