#metabrainz

The passbolt password manager has a release song for every release. I love that. And a password manager is totally unrelated to music. We should have this for MB releases as well :)

Their current release song is https://www.youtube.com/watch?v=zUzd9KyIDrM , as announced in https://www.passbolt.com/changelog/api-bext/b-y...

hi Jade (I have the time right, right? 🙂)

bitmap[m]: Yep!

Waiting for my PC to boot up

cool, I won't keep you long :)

FYI I've deployed the mb-mail-service to aretha (where test.mb lives) and deployed your MBS branch there too

but I think I need to set some DBDefs values before we can try it

Yep, that's likely

My musicbrainz-server has a slightly modified Default.pm file cause I couldn't figure out which was the right DBDefs.pm to modify last week

sub SMTP_SERVER { 'smtp-relay:1025' }

sub MAIL_SERVICE_BASE_URL { "http://mail-service:3000" }

lib/DBDefs.pm is the one you want to modify (to override anything in Default.pm)

if that doesn't exist there's a DBDefs.pm.sample you can base it on

And I've got this added to the docker compose file... (full message at <https://matrix.chatbrainz.org/_matrix/media/v3/...>)

Did you have any issues with the Sentry integration?

I haven't configured that yet actually, because I think our Sentry instance is still borked 😔

It's a bit difficult for me to test that locally with sentry going on about free trials and stuff 😅

It was nagging me about uploading debuginfo, but I couldn't actually find any docs for it

heh, so far it's been even more difficult to test in prod because it doesn't want to work at all

It feels like there's a certain category of software that's just way too complex to operate. GitLab fits too

yeah, sentry feels crazy bloated compared to what I remember from years ago

which endpoints did you implement for sending out mail so far?

(in MBS)

Email verification and message sending from last week

I haven't been too productive this week :(

A bit distracted by something I'd best no talk about in public here, but which could be very good

I was also meant to be packing today, was mostly procrastinating

s/no/not/

that's alright, there's not a whole lot left to your project at this point:)

ok, I'll give email verification emails a test today

I'm very happy that it's gone so well so far 😁

bitmap[m]: I've had a few issues with the strings in that just being empty for some reason??

hmm, weird. I'll keep an eye out for that

At first I thought it was because it was a blessed object or something

But I added the to_string line https://github.com/JadedBlueEyes/musicbrainz-se...

did that help with anything?

I wouldn't expect a blessed object there either

And it fixed it throwing, but still I'm getting an empty string, now in the name field too

* And it fixed it throwing without the string concatenation, but still I'm getting an empty string, now in the name field too

If I tried to give $verification_link directly to the JSON encoder, it would throw

Forcing it to be a string in line 437 fixes that, or what I added in line 424

ok, sounds like something is being passed incorrectly, since it shouldn't be an object

(the JSON encoder doesn't encode blessed objects by default)

I'll look into it and ping you if I find something

Thanks :)

I also think there's probably something I should be doing other than... (full message at <https://matrix.chatbrainz.org/_matrix/media/v3/...>)

yeah, we should probably die there (I think the controller will handle that correctly and show an error to the user, but should confirm that)

at least for email verification, for stuff like subscriptions we probably don't want to abort the whole process

The request should always return 200 for bulk sending anyways

ah, right

I think that should be everything for this week?

my initial test didn't send out an email but I almost definitely misconfigured things, I'll figure it out :)

Did some small things, mf1 is now from crates.io: https://github.com/metabrainz/mb-mail-service/c...

yeah, nothing else from me until I test the test deployment and review the Perl issues

bitmap[m]: It took me way too long to get everything configured right, it was mostly making sure the settings were actually *set* properly

Jade[m]: Slowly moving away from git dependencies

heh, I think I've set them to the wrong values properly in this case

anyway :) I'll let you get going, thanks for your time!

No problem, see you next week 😁

Oh this was an actual line in my shell history this week:

hah, what's dnf?

Fedora's package manager

oh! I haven't used Fedora in ages, it used to be yum?

Yeah it's changed a bit since then

they're working on dnf5 at the moment, which is even more of an improvement

Jade[m]: Installing almost all the musicbrainz-server perl modules so that the perl language server works was the cause of this monstrosity

[bookbrainz-site] 14yourselfthere opened pull request #1110 (03master…master): just learning https://github.com/metabrainz/bookbrainz-site/p...

Hello aerozol !

When you have a minute, please let me know what you think of LB#2956 for multi-track-linking (deployed on https://test.listenbrainz.org/settings/missing-...). I deviated from your plans on the associated ticket, which is signifcantly harder to implement. I wanted to try a simple version first :)

LB-1281: Link all listens from the same album: https://github.com/metabrainz/listenbrainz-serv...

Jade: got the verification emails to send out from test.mb 🎉 but yeah, the strings are empty

regarding verification_link being a blessed object, it is indeed a URI instance, which makes sense now. URI should overload "" already, so stringifying it like "$verification_link" should be sufficient to work around that

That's one of the first things I tried, but it seemed to result in an empty string 😕

huh, it works for me 🤷‍♂️

the empty variables in the rendered mail seems to be on the Rust side of things though (since the to_name and verification_url parameters are being sent/received correctly)

<bitmap[m]> "the empty variables in the..." <- I'll have a deeper look into it when I get an opportunity

hi, i tried init'ing mbslave today but came across a sql constraint error. not sure how to resolve...

psycopg2.errors.CheckViolation: new row for relation "label" violates check constraint "label_label_code_check"

DETAIL: Failing row contains (294731, dacec7dc-806e-4f1a-ab41-cc0c46b297e0, beau by Republic, 2022, 10, 9, null, null, null, 202210, 1, 7741, , 0, 2024-05-14 21:06:04.842073+00, f).

you have the latest mblsave code? https://github.com/acoustid/mbslave/commits/main/ schema 29 should have the necessary label_label_code_check changes

i installed mbslave using pipx

is there a better way?

bttf (IRC): looks like v29 was never published to pypi. but I think you can pass a git URL to pipx

ah ok. thanks

pipx install git+https://github.com/acoustid/mbslave.git might work

trying it out now

for me it still says "installed package mbslave 28.0.0" but I think that's just because the version wasn't updated in pyproject.toml

looks like it's working. thx

monkey: I saw the LB#2956 updates, I’m very excited!! Just had a crazy week. Today I have time though, will look at it thoroughly in an hour or two

LB-1281: Link all listens from the same album: https://github.com/metabrainz/listenbrainz-serv...

<04dieseltech> Was told to post here for my problem: I'm getting a 504 error when trying to view my profile on LB It seems people on the forums have this issue from time to time and someone clears out the cache on the backend for it to start working again If you wait long enough for the page to load this comes up