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

Relays intermittently deauthed even though "always" is selected #234

Open
laanwj opened this issue Sep 19, 2024 · 4 comments
Open

Relays intermittently deauthed even though "always" is selected #234

laanwj opened this issue Sep 19, 2024 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@laanwj
Copy link
Contributor

laanwj commented Sep 19, 2024

i'm not sure if this is a problem with the relays, or a bug in nostrudel, or a combination, resulting in this behavior, but some relays tend to get de-authed intermittently. They then have to be manually authed again by clicking the lock.

For example:

Still happens with: v0.41.0

cc: @SilberWitch

@SilberWitch
Copy link

Yup, same with me and some other npubs.

@hzrd149
Copy link
Owner

hzrd149 commented Sep 25, 2024

This is probably caused by a few things

  1. noStrudel will try and close relay connections its no longer using ( even if you are authenticated ).
  2. noStrudel waits for an auth-required: error message from the relay before it tries to authenticate, even if its set to always

So if the relay connection was closed in the background, then reopened. it wouldn't authenticate until the relay send the auth error

There are a few ways we could solve this

  1. Don't close connections to authenticated relays
  2. Proactively authenticate to relays that are set to always

@hzrd149 hzrd149 added the enhancement New feature or request label Sep 25, 2024
@hzrd149 hzrd149 self-assigned this Sep 25, 2024
@hzrd149 hzrd149 moved this to Alpha Version in noStrudel Sep 25, 2024
@hzrd149
Copy link
Owner

hzrd149 commented Sep 25, 2024

Add both options to next version bbd19d7

image

@SilberWitch
Copy link

Ah, nice. Yes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Alpha Version
Development

No branches or pull requests

3 participants