#metabrainz

/

      • minimal has quit
      • Maxr1998 has quit
      • Maxr1998 joined the channel
      • aerozol[m] has quit
      • pite has quit
      • Kladky joined the channel
      • tarun joined the channel
      • tarun has quit
      • outsidecontext[m has quit
      • d4rk-ph0enix joined the channel
      • d4rkie has quit
      • AJ35Services[m] has quit
      • outsidecontext[m joined the channel
      • outsidecontext[m
        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 :)
      • Sophist-UK joined the channel
      • pite joined the channel
      • minimal joined the channel
      • bitmap[m]
        hi Jade (I have the time right, right? πŸ™‚)
      • Jade[m] joined the channel
      • Jade[m]
        bitmap[m]: Yep!
      • Waiting for my PC to boot up
      • bitmap[m]
        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
      • Jade[m]
        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" }
      • bitmap[m]
        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
      • Jade[m]
        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?
      • bitmap[m]
        I haven't configured that yet actually, because I think our Sentry instance is still borked πŸ˜”
      • Jade[m]
        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
      • bitmap[m]
        heh, so far it's been even more difficult to test in prod because it doesn't want to work at all
      • bitmap[m] set MAIL_SERVICE_BASE_URL on test.mb just now
      • Jade[m]
        It feels like there's a certain category of software that's just way too complex to operate. GitLab fits too
      • bitmap[m]
        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)
      • Jade[m]
        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/
      • bitmap[m]
        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
      • Jade[m]
        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??
      • bitmap[m]
        hmm, weird. I'll keep an eye out for that
      • Jade[m]
        At first I thought it was because it was a blessed object or something
      • bitmap[m]
        did that help with anything?
      • I wouldn't expect a blessed object there either
      • Jade[m]
        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
      • bitmap[m]
        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
      • Jade[m]
        Thanks :)
      • I also think there's probably something I should be doing other than... (full message at <https://matrix.chatbrainz.org/_matrix/media/v3/...>)
      • bitmap[m]
        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
      • Jade[m]
        The request should always return 200 for bulk sending anyways
      • bitmap[m]
        ah, right
      • Jade[m]
        I think that should be everything for this week?
      • bitmap[m]
        my initial test didn't send out an email but I almost definitely misconfigured things, I'll figure it out :)
      • Jade[m]
        Did some small things, mf1 is now from crates.io: https://github.com/metabrainz/mb-mail-service/c...
      • bitmap[m]
        yeah, nothing else from me until I test the test deployment and review the Perl issues
      • Jade[m]
        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
      • bitmap[m]
        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!
      • Jade[m]
        No problem, see you next week 😁
      • Oh this was an actual line in my shell history this week:
      • Jade[m] uploaded an image: (222KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/matrix.org/XYyfchMMnxUumWMDUHYMaRXU/image.png >
      • bitmap[m]
        hah, what's dnf?
      • Jade[m]
        Fedora's package manager
      • bitmap[m]
        oh! I haven't used Fedora in ages, it used to be yum?
      • Jade[m]
        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
      • BrainzGit
        [bookbrainz-site] 14yourselfthere opened pull request #1110 (03master…master): just learning https://github.com/metabrainz/bookbrainz-site/p...
      • monkey[m]
        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 :)
      • BrainzBot
        LB-1281: Link all listens from the same album: https://github.com/metabrainz/listenbrainz-serv...
      • thevar1able has quit
      • thevar1able joined the channel
      • hirigaray has quit
      • hirigaray joined the channel
      • hirigaray has quit
      • hirigaray joined the channel
      • bitmap[m]
        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
      • Jade[m]
        That's one of the first things I tried, but it seemed to result in an empty string πŸ˜•
      • minimal has quit
      • bitmap[m]
        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)
      • Jade[m]
        <bitmap[m]> "the empty variables in the..." <- I'll have a deeper look into it when I get an opportunity
      • bttf joined the channel
      • bttf
        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).
      • bitmap[m]
        you have the latest mblsave code? https://github.com/acoustid/mbslave/commits/main/ schema 29 should have the necessary label_label_code_check changes
      • bttf has quit
      • bttf joined the channel
      • bttf
        i installed mbslave using pipx
      • is there a better way?
      • nawcom has quit
      • nawcom joined the channel
      • bitmap[m]
        bttf (IRC): looks like v29 was never published to pypi. but I think you can pass a git URL to pipx
      • bttf
        ah ok. thanks
      • bitmap[m]
      • bttf
        trying it out now
      • bitmap[m]
        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
      • bttf
        looks like it's working. thx
      • Kladky has quit
      • aerozol[m] joined the channel
      • aerozol[m]
        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
      • BrainzBot
        LB-1281: Link all listens from the same album: https://github.com/metabrainz/listenbrainz-serv...
      • bttf has quit
      • discordbrainz
        <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