Hello! I am trying to setup LB server on my local machine. When I try to connect to Spotify, I get 'INVALID_CLIENT: Invalid redirect URI' . I used 'https://localhost:7000/profile/music-services/spotify/callback/ ' as the callback url while registering the spotify app
Have also tried 'https://localhost/profile/music-services/spotify/callback/ ' as the callback uri
[listenbrainz-server] 14mayhem opened pull request #1780 (03year-in-music…year-in-music-add-playlists): Add support for importing troi generated playlist excerpts https://github.com/metabrainz/listenbrainz-serv...
ShivamAwasthi
alastairp Now when I try to connect, it shows 'Unable to connect'
alastairp
that sounds like a good start - now spotify is happy, but it seems that maybe something is wrong with your local server
we did just make some changes here recently - one moment, I'm setting it up again myself to verify that there's not an unexpected error that we introduced
the unable to connect error - is that an error from your browser?
I think maybe there is some mixup because of documentation not up-to-date? Like the documentation asks us to go for port 80 by default, but it has been updated to port 7000
alastairp
ShivamAwasthi: right, we recently change to 7000, because 80 was giving us some problems during testing. Where did you find documentation that still talks about 80? We should update that if it still has the old values
Thanks yvanzo. We're EC2 based so we don't use Docker. I'll see if I can't provide a PR to exclude log4j-api < 11.15.0 and include 11.15.0 as Solr don't seem to have patched it yet, if that's useful to you?
Thanks yvanzo. We're EC2 based so we don't use Docker. I'll see if I can't provide a PR to exclude log4j-api < 2.15.0 and include 2.15.0 as Solr don't seem to have patched it yet, if that's useful to you?
Sorry - got version numbers wrong. my memory is poor.
yvanzo
mglubb: you can also pass SOLR_OPTS="-Dlog4j.formatMsgNoLookups=true" for now
mglubb
I understand, yvanzo but that doesn't fix your declared deps and relies on educated users
but every time I run it, no data gets written to the DB.
lucifer
mglubb: fwiw, SOLR has now released official patched images and they use the same workaround to pass the system property.
mayhem
when it clearly should, imho. can you sanity check the query for me please.
lucifer
yes will do in a few mins.
mayhem
thx
mglubb
Thanks lucifer. We don't run under Docker though. s'ok, I'll set Solr opts for now.
lucifer
ah right, i didn't fully understand your issue at first 😅
reosarevok
yvanzo, lucifer: can we now update docker to use the official patched images then?
yvanzo
reosarevok: we don't use official images, and Solr 7 isn’t patched either.
lucifer
reosarevok: we probably can but its not straight forward. we have a fork of solr images repo and there are extra commits in our repo. will need to rebuild a couple of base images and update them if needed and then use that in mb docker.
mglubb
That's right. I'm surprised Apache haven't released an official version with the log4j dependency raised
yvanzo
We plan to move to Solr 8 eventually which is going to be fixed but has only mitigation steps for now.
lucifer
while log4j api doesn't change frequently, there are various changes in point version so some widely used projects are holding off updating version and instead setting the property or env var.
also solr 7 is unmaintained so it won't receive an update. solr 8 will
PopperBruda has left the channel
mayhem: query looks correct so not sure whats wrong but i have seen this happen before where no error is logged but data isn't inserted either. where is the dump file available, i'll try to debug it.
Others up for review today: reosarevok, yvanzo, bitmap, akshat, zas, ruaok, monkey, lucifer, alastairp, Freso – anyone else who wants to give review, let me know ASAP.
reosarevok: Go!
reosarevok
Hi!
zas
bitmap, reosarevok : IP blocked
reosarevok
I worked on updating open PRs to the latest master and fixing eslint / Perl::Critic issues found by new rules
TOPIC: MetaBrainz Community and Development channel | MusicBrainz non-development: #musicbrainz | BookBrainz: #bookbrainz | Channel is logged; see https://musicbrainz.org/doc/IRC for details | Agenda: Reviews, MB team ignoring telegram (ruaok), upcoming meeting(s) (Freso) [postponed: gravatar (MD5 adresses) (cat)]
