Account switching does not work correctly #530

New Issue

Kieran · 2023-04-19T23:25:04Z

mattn commented

2023-04-19 23:25:04 +00:00

(Migrated from github.com)

Describe the bug

An error has occured!
{}

To Reproduce
Steps to reproduce the behavior:

Go to 'https://snort.social'
Click on 'Logout'
Login with NIP-07
See that relay informations are empty
Add new relay once
See an error

Expected behavior

No errors

Desktop (please complete the following information):

OS: Linux
Browser: chrome
Version: latest (I can't confirm because the error)

Smartphone (please complete the following information):

I can't repro this on mobile phone

Additional context

As far as I can look, relay informations are not restored with logins.

**Describe the bug** ``` An error has occured! {} ``` **To Reproduce** Steps to reproduce the behavior: 1. Go to 'https://snort.social' 2. Click on 'Logout' 3. Login with NIP-07 4. See that relay informations are empty 5. Add new relay once 6. See an error **Expected behavior** No errors **Desktop (please complete the following information):** - OS: Linux - Browser: chrome - Version: latest (I can't confirm because the error) **Smartphone (please complete the following information):** I can't repro this on mobile phone **Additional context** As far as I can look, relay informations are not restored with logins.

mattn commented

2023-04-20 08:06:03 +00:00

(Migrated from github.com)

Hmm, it seems that this occur only with my npub. Could you please login with my npub?

npub1937vv2nf06360qn9y8el6d8sevnndy7tuh5nzre4gj05xc32tnwqauhaj6

Hmm, it seems that this occur only with my npub. Could you please login with my npub? npub1937vv2nf06360qn9y8el6d8sevnndy7tuh5nzre4gj05xc32tnwqauhaj6

mattn commented

2023-04-20 12:58:53 +00:00

(Migrated from github.com)

I tried to reproduce this with "yarn run start". But not reproduced.

v0l commented

2023-04-20 13:15:43 +00:00

(Migrated from github.com)

I see the relays are not updating properly for other accounts, multi-account still needs a bit of work

mattn commented

2023-04-23 17:33:10 +00:00

(Migrated from github.com)

I figure out why this occured.

Few weeks ago, I send DM to my self via my CLI tool. The message was broken as that kind is 4 but tags is empty. So snort raise an error when load DMs. Probably, recent changes make snort restrictly. I tried to delete the DM but wss://relay.snort.social does not behave deletion of DM with kind 5.

Could you please handle which one of two.

support kind 5 for DM on relay.snort.social
ignore broken DMs (ex: kind 4 but tags is empty)

I figure out why this occured. Few weeks ago, I send DM to my self via my CLI tool. The message was broken as that kind is 4 but tags is empty. So snort raise an error when load DMs. Probably, recent changes make snort restrictly. I tried to delete the DM but wss://relay.snort.social does not behave deletion of DM with kind 5. Could you please handle which one of two. 1. support kind 5 for DM on relay.snort.social 2. ignore broken DMs (ex: kind 4 but tags is empty)

mattn commented

2023-04-25 00:24:50 +00:00

(Migrated from github.com)

Now I noticed that wss://relay.snort.social become payed relay. I can no longer log in to snort unless I delete the 4 broken DMs. please help @v0l

mattn commented

2023-04-25 00:44:05 +00:00

(Migrated from github.com)

FYI, This unwrap raise an exception when the DM does not have any tags.

9258253f65/packages/app/src/Pages/MessagesPage.tsx (L107-L108)

FYI, This unwrap raise an exception when the DM does not have any tags. https://github.com/v0l/snort/blob/9258253f65583763793dae57fcc51d2f4d135a96/packages/app/src/Pages/MessagesPage.tsx#L107-L108

Kieran referenced this issue from a commit

2023-05-04 13:17:08 +00:00

bug: ignore DM's with no `p` tag

Sign in to join this conversation.