A diet Tap option for handling atproto sync
atclient: allow pw auth to inactive account (#1281)
This check was overly aggressive. There are a lot of situations where an
account needs to authenticate in "deactivated" state.
I'm not even sure if a log line is necessary here, but figure this is a
change in SDK behavior so it should at least be a bit vocal.
This also fixes a bug where a pointer to the status string was printed,
instead of the status string itself.
+2
-1
+2
-1
atproto/atclient/password_auth.go
+2
-1
atproto/atclient/password_auth.go
···
4
4
"context"
5
5
"encoding/json"
6
6
"fmt"
7
+
"log/slog"
7
8
"net/http"
8
9
"strings"
9
10
"sync"
···
266
267
}
267
268
268
269
if out.Active != nil && *out.Active == false {
269
-
return nil, fmt.Errorf("account is disabled: %v", out.Status)
270
+
slog.Info("password login to inactive account", "status", *out.Status, "username", username)
270
271
}
271
272
272
273
did, err := syntax.ParseDID(out.Did)