MetaBrainz

Room ID: !hyuEWFORfWYLZitABm:chatbrainz.org

BrainzGit (IRC)
[troi-recommendation-playground] Mohamedselim20 closed pull request #169 (main…main): LB-1504 Added a column header using PrettyTable https://github.com/metabrainz/troi-recommendation-playground/pull/169
BrainzGit (IRC)
[troi-recommendation-playground] Mohamedselim20 opened pull request #171 (main…patch-1): LB-1504 : Update print_recording.py https://github.com/metabrainz/troi-recommendation-playground/pull/171
Jade
Happy Easter!
Unsupported message type m.room.member
Unsupported message type m.room.member
bitmap
Reacted 🥚 to event $9SLihGpkfj5lj2PpbbxI7RjNBgSjD83ya-u1nBLYTXw
Bob Swift
Reacted 🥚 to event $9SLihGpkfj5lj2PpbbxI7RjNBgSjD83ya-u1nBLYTXw
bitmap
sorry for the delay, I've tested signing in now. getting into Kandim with my passkey works fine; when I click on "Google (SAML)" from the applications page, it takes me to Google's "Verify it’s you" page, but hitting "Next" from there just redirects me back to https://auth.metabrainz.org/ui/apps like nothing happened
bitmap
PM'd you the SAML tracer logs
julian45
hmm
julian45
i will look at those in a bit - how about signing into gmail with a fresh session?
bitmap
I get the same behavior in an incognito window (Firefox) and in Safari
julian45
🤔
BrainzGit (IRC)
[musicbrainz-server] mwiencek opened pull request #3522 (master…mbs-13987): MBS-13987: Fix seeding ISRC/ISWC fields https://github.com/metabrainz/musicbrainz-server/pull/3522
kellnerd
Reacted 🥚 to event $9SLihGpkfj5lj2PpbbxI7RjNBgSjD83ya-u1nBLYTXw
BrainzGit (IRC)
[listenbrainz-ios] 07jasjeet opened pull request #33 (main…fix-feed): Fix feed https://github.com/metabrainz/listenbrainz-ios/pull/33
BrainzGit (IRC)
[listenbrainz-ios] 07jasjeet merged pull request #33 (main…fix-feed): Fix feed https://github.com/metabrainz/listenbrainz-ios/pull/33
akshaaatt
jasje: do not push code to the main branch directly since there are triggers attached to it and it sends a build across.
jasje
akshaaatt: can we disable it?
akshaaatt
The org has limited hours used for such triggers
jasje
I just realized this
akshaaatt
It makes less sense to disable it. Why don’t you work on a dev branch?
akshaaatt
Reacted 👍 to event $xKePLy7fXP2Bd5Ceew5huCZUf-jqmpYAPLEBbJJXDms
jasje
after the build number was bumped significantly
jasje
I push updates my self.
jasje
when things are ready
jasje
so why not
akshaaatt
Why would you do that? CI CD is the best to deploy
akshaaatt
Access control is messed up so it’s better not to rely on individuals for deployments
akshaaatt
The solution I see to this is a dev branch. I had also made dev and qa schemes to use for those btw
jasje
I don't see any github actions on the repo
jasje
where is the code being build and pushed from?
akshaaatt
It’s not gh actions. It’s xcode cloud actions. Configured differently
jasje
i see
akshaaatt
Yus
jasje
* where is the code being built and pushed from?
tux0r (IRC)
lol git.
Unsupported message type m.room.member
monkey
Reacted 👌 to event $mwIulNYfcKwhKPF58Y4gmNMA1LQkEmYa4HsdhcX_1ZY
BrainzGit (IRC)
[listenbrainz-server] MonkeyDo merged pull request #3253 (master…LB-1776--Artist-origin-map-tidy-up): LB-1776: Artist origin map tidy up https://github.com/metabrainz/listenbrainz-server/pull/3253
Unsupported message type m.room.member
Unsupported message type m.room.member
Unsupported message type m.room.member
@fettuccinae:matrix.org
lucifer: To implement this ticket, should I extend the listens_importer table to include the status of importer which will be updated by the lastfm importer and frontend will check this to update the user ( the way we do it in export.py).
BrainzBot (IRC)
LB-1768: Show LastFM importer status or percentage
@fettuccinae:matrix.org
* lucifer: To implement this ticket, should I extend the listens_importer table to include the status of importer which will be updated by the lastfm importer and frontend will check it to update the user ( the way we do it in export.py).
Jade
Are we having a meeting today?
Jade
Can't remember 😅
Bob Swift
I don't think so.
julian45
per the decision reached at the end of last meeting, we're not meeting today
julian45
just made some changes to the google side, could you please try again in a fresh session?
julian45
(misread the field names in google's config and realized I was telling it to send saml auth requests to a generic page, rather than the endpoint on the saml proxy meant for consuming the request — that's what i get for trying to mess with saml at night, lol)
bitmap
it's different now, at least :) but when I log in from Google it redirects me to a plain text page with https://accounts.google.com/samlrp/<random string> visible and gets stuck there
Jade
Reacted 👍️ to event $ZNSqBAdPG9MhRSKn2wCcU3kQ-Xn74i6vRpmhSu_lPM0
bitmap
(stuck on the /kanidm_saml-google_oidc/sso/redirect page, that is. also, this page returns a 500 status code)
julian45
f#$(king SAML weirdness... there are two possible types of request that an SP can send to start a saml auth request and the saml proxy has two different endpoints for them... i guessed one, maybe it was the other (i'll also check the proxy's logs in a bit
julian45
made another change (trying the other endpoint), please give it a shot in a few
bitmap
getting the same issue
bitmap
(should the /kanidm_saml-google_oidc/sso/post endpoint have changed? because I still end up there)
julian45
i expected the change you seem to have seen (post instead of redirect) but not the same behavior :/ can you please send me a SAML-tracer export with another attempt?
bitmap
Reacted 👍️ to event $EF75Ci1OC3BPkstpsl9rTW4zs2b4-eRxnOG2saWTVHs
bitmap
sent, hope it helps!
julian45
if no further fiddling works, luckily google has another SSO option (OIDC) in beta that should be a good deal less fiddly but we do need saml working for at least one or two other services :/
BrainzGit (IRC)
[bookbrainz-site] dependabot[bot] opened pull request #1165 (master…dependabot/npm_and_yarn/elastic/elasticsearch-9.0.0): chore(deps): bump @elastic/elasticsearch from 5.6.22 to 9.0.0 https://github.com/metabrainz/bookbrainz-site/pull/1165
BrainzGit (IRC)
[bookbrainz-site] dependabot[bot] closed pull request #1155 (master…dependabot/npm_and_yarn/elastic/elasticsearch-8.17.1): chore(deps): bump @elastic/elasticsearch from 5.6.22 to 8.17.1 https://github.com/metabrainz/bookbrainz-site/pull/1155
Unsupported message type m.room.member
Unsupported message type m.room.member
lucifer
@fettuccinae:matrix.org: yup that sounds good to me.
@fettuccinae:matrix.org
Reacted 👍️ to event $9MDFGm6wng2bOX9f2BjTrKaI54gvneyl2kknuKkVf14
lucifer
mayhem: did you checkout the mlhd similarity data on labs api?
mayhem
not yet. I will try and do that today, but today is filled with heavy shit.
lucifer
Reacted 👍️ to event $KLdScBiH7LptNzc5wdYkHYRB0Qk20TY-KClVbriq3R8
julian45
fixed a config bit on the saml proxy side, please try again (w/ trace) when you get a chance. i went to try it myself but i think bc i have some admin rights in the console it forces google mfa on me anyway
mayhem
I have some very sad new to share: https://blog.metabrainz.org/2025/04/22/rest-in-peace-drsaunde/
zas
Reacted 🫂 to event $aFZxI5J6OftXBH4AJXLjdCexaZY4eCZnCgS_plPMK1c
monkey
Reacted 😔 to event $aFZxI5J6OftXBH4AJXLjdCexaZY4eCZnCgS_plPMK1c
kellnerd
Reacted 😔 to event $aFZxI5J6OftXBH4AJXLjdCexaZY4eCZnCgS_plPMK1c
mayhem
I took a very quick look, which is all I have time for today, but ideally similar recordings would not show recordings by the same artists....
lucifer
Example of where it happens
lucifer
*
Jade
Reacted 😔 to event $aFZxI5J6OftXBH4AJXLjdCexaZY4eCZnCgS_plPMK1c
mayhem
oh sorry, my bad. ignore that. (#toomuchontoday)
Unsupported message type m.room.member
@fettuccinae:matrix.org
Reacted 😔 to event $aFZxI5J6OftXBH4AJXLjdCexaZY4eCZnCgS_plPMK1c
bitmap
Reacted 😔 to event $aFZxI5J6OftXBH4AJXLjdCexaZY4eCZnCgS_plPMK1c
bitmap
Reacted 🫂 to event $aFZxI5J6OftXBH4AJXLjdCexaZY4eCZnCgS_plPMK1c
bitmap

the flow was slightly different now, kandim actually asked me for permission to read my email address, though now I end up on a google page that says:

Couldn’t sign you in
Contact your domain admin for help.

bitmap
sent the latest SAML logs
BrainzGit (IRC)
[musicbrainz-server] mwiencek merged pull request #3522 (master…mbs-13987): MBS-13987: Fix seeding ISRC/ISWC fields https://github.com/metabrainz/musicbrainz-server/pull/3522
julian45
that's promising! i'll look at the traces you sent, but my guess is that i need to update the attribute mappings in the proxy since the full principal names for users in kanidm =/= emails -- this would be a non issue in other idm tools (including the secondary candidate, authentik) where the principal names can end in different domains from the server host (plus authentik supports SAML natively, which sidesteps the whole proxy mess)
bitmap
Reacted 🤞 to event $4eGSY025lVQ-D_sWVn_g4q6xEm_ndWrRPl51jN4sdcA
julian45
main reasons we're trying kanidm first is it's super lightweight + theoretically can handle linux server login and stuff like ssh key distribution natively, which authentik can't; but, before we can get to the nice stuff like that, we have to validate use cases that would be applicable regardless of tool
Unsupported message type m.room.member
BrainzGit (IRC)
[musicbrainz-server] mwiencek merged pull request #3516 (master…other_bubbles): MBS-13980: Add artist, label, and place doc bubbles (I) https://github.com/metabrainz/musicbrainz-server/pull/3516
aerozol
Reacted 😔 to event $aFZxI5J6OftXBH4AJXLjdCexaZY4eCZnCgS_plPMK1c
BrainzGit (IRC)
[musicbrainz-server] reosarevok merged pull request #3521 (master…MBS-13985): MBS-13985: Show inline primary alias for work authors https://github.com/metabrainz/musicbrainz-server/pull/3521
BrainzGit (IRC)
[musicbrainz-server] reosarevok merged pull request #3519 (master…extra-wording-for-mbs-13711): Add short note about exceptions to new capitalization warning https://github.com/metabrainz/musicbrainz-server/pull/3519
BrainzGit (IRC)
[musicbrainz-server] reosarevok merged pull request #3517 (master…MBS-13982): MBS-13982: Wrap overlong release and release group titles in tables https://github.com/metabrainz/musicbrainz-server/pull/3517
BrainzGit (IRC)
[musicbrainz-server] reosarevok merged pull request #3518 (master…MBS-13977): MBS-13977: Ignore country codes in yesasia.com links https://github.com/metabrainz/musicbrainz-server/pull/3518
Jade
Prototype for the archiver is coming along well, by the way