lucifer: ahhh, yes. I was confused about that last night.
2025-02-17 04846, 2025
mayhem[m]
lucifer: also a regression has snuck in -- new user logins are no longer working. the login_id column doesn't get set anymore.
2025-02-17 04850, 2025
HemangMishra[m] has quit
2025-02-17 04813, 2025
julian45[m] joined the channel
2025-02-17 04814, 2025
julian45[m]
monkey: zas : since you two commented on the doc I introduced in last week's meeting, just wanted to let you know that I made some changes to it^, so I'd appreciate it if you could take a second look
2025-02-17 04814, 2025
julian45[m]
^Added a proposal section before the tool descriptions, changed tool rankings, added some details to one of the tool descriptions
lucifer: it's already present in listenbrainz db, do you mean create it in mb ?
2025-02-17 04834, 2025
lucifer[m]
mayhem: the user.login_id = str(uuid.uuid4()) is not needed. i think if you just delete your local user table and recreate it everything should work fine
2025-02-17 04823, 2025
lucifer[m]
setting login_id for existing users will cause the user to logout from all other sessions. for instance, if you log in on chrome you get logged out on firefox etc.
2025-02-17 04803, 2025
mayhem[m]
I did recreate everything and it didn't work.
2025-02-17 04810, 2025
mayhem[m]
I can try it again.
2025-02-17 04818, 2025
lucifer[m]
mayhem: the if check inside the peewee.DoesNotExist looks wrong.
2025-02-17 04835, 2025
lucifer[m]
you are disallowing any non-admin user from creating an account
2025-02-17 04800, 2025
lucifer[m]
do you want to restrict registration on the server?
2025-02-17 04823, 2025
mayhem[m]
only users who have a (blank) row in the user table are allowed to log in.
2025-02-17 04833, 2025
mayhem[m]
account creation is entirely manual by the admin.
2025-02-17 04809, 2025
lucifer[m]
i see. so you need to set login_id when creating the blank row.
2025-02-17 04841, 2025
mayhem[m]
oki.
2025-02-17 04814, 2025
lucifer[m]
do you have an admin panel view to create users yet or should i add it and set it up?
2025-02-17 04842, 2025
mayhem[m]
I have it.
2025-02-17 04827, 2025
mayhem[m]
what would be a good way to add this UUID for when the user is created?
2025-02-17 04851, 2025
mayhem[m]
since the admin view is automated, I can't control the creation of records.
2025-02-17 04830, 2025
BrainzGit
[listenbrainz-server] 14MonkeyDo merged pull request #3166 (03master…duplicate-notify): LB-1742 Notify user if they're adding a duplicate to a playlist https://github.com/metabrainz/listenbrainz-server…
2025-02-17 04826, 2025
lucifer[m]
gimme a sec
2025-02-17 04818, 2025
lucifer[m]
pushed changes, mayhem
2025-02-17 04829, 2025
lucifer[m]
i set the default in the peewee model itself.
2025-02-17 04854, 2025
mayhem[m]
thank you!!
2025-02-17 04851, 2025
mayhem[m] uploaded an image: (11KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/chatbrainz.org/ekTrEqmdWwrOvMWgARyfxDEa/image.png >
2025-02-17 04855, 2025
mayhem[m]
lolololol
2025-02-17 04809, 2025
reosarevok[m] joined the channel
2025-02-17 04810, 2025
reosarevok[m]
Is that field enabled when visiting from Germany?
2025-02-17 04830, 2025
monkey[m] joined the channel
2025-02-17 04831, 2025
monkey[m]
Faxe number = 10%
2025-02-17 04831, 2025
monkey[m] uploaded an image: (936KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/chatbrainz.org/mPNUqiLQqpkbEGliDpLopdrk/image.png >
2025-02-17 04839, 2025
monkey[m]
(and yes, this is a 1 liter can)
2025-02-17 04825, 2025
lucifer[m]
salaxceitor: the table needs to be in listenbrainz timescale db
2025-02-17 04845, 2025
salaxceitor[m]
lucifer: let me check
2025-02-17 04852, 2025
salaxceitor[m]
it's present in listenbrainz_ts -> public -> background_worker_state
lucifer or mayhem : can one of you run the sql script that came with the "pause account" PR please? Currently getting 500 error when accessing beta as the column is missing in the DB
2025-02-17 04804, 2025
mayhem[m]
k
2025-02-17 04811, 2025
monkey[m]
(sorry, having food)
2025-02-17 04839, 2025
mayhem[m]
done.
2025-02-17 04855, 2025
monkey[m]
Updating LB prod
2025-02-17 04835, 2025
BrainzGit
[listenbrainz-server] 14dependabot[bot] opened pull request #3182 (03master…dependabot/npm_and_yarn/dompurify-3.2.4): Bump dompurify from 2.5.8 to 3.2.4 https://github.com/metabrainz/listenbrainz-server…
2025-02-17 04842, 2025
monkey[m]
^ Gotta review all uses of Dompurify in the PR before merging, just FYI
[musicbrainz-server] 14mwiencek merged pull request #3467 (03master…mbs-13925): MBS-13925: Instrument name erroneously displayed in link type autocomplete https://github.com/metabrainz/musicbrainz-server/…
