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
2021-12-13 34738, 2021
ShivamAwasthi
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-server…
2021-12-13 34718, 2021
ShivamAwasthi
alastairp Now when I try to connect, it shows 'Unable to connect'
2021-12-13 34759, 2021
alastairp
that sounds like a good start - now spotify is happy, but it seems that maybe something is wrong with your local server
2021-12-13 34721, 2021
alastairp
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
2021-12-13 34736, 2021
alastairp
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
2021-12-13 34742, 2021
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?
2021-12-13 34716, 2021
mglubb
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?
2021-12-13 34730, 2021
mglubb
Sorry - got version numbers wrong. my memory is poor.
2021-12-13 34735, 2021
yvanzo
mglubb: you can also pass SOLR_OPTS="-Dlog4j.formatMsgNoLookups=true" for now
2021-12-13 34704, 2021
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.
2021-12-13 34728, 2021
lucifer
mglubb: fwiw, SOLR has now released official patched images and they use the same workaround to pass the system property.
2021-12-13 34738, 2021
mayhem
when it clearly should, imho. can you sanity check the query for me please.
2021-12-13 34750, 2021
lucifer
yes will do in a few mins.
2021-12-13 34712, 2021
mayhem
thx
2021-12-13 34752, 2021
mglubb
Thanks lucifer. We don't run under Docker though. s'ok, I'll set Solr opts for now.
2021-12-13 34722, 2021
lucifer
ah right, i didn't fully understand your issue at first 😅
2021-12-13 34703, 2021
reosarevok
yvanzo, lucifer: can we now update docker to use the official patched images then?
2021-12-13 34722, 2021
yvanzo
reosarevok: we don't use official images, and Solr 7 isn’t patched either.
2021-12-13 34728, 2021
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.
2021-12-13 34751, 2021
mglubb
That's right. I'm surprised Apache haven't released an official version with the log4j dependency raised
2021-12-13 34731, 2021
yvanzo
We plan to move to Solr 8 eventually which is going to be fixed but has only mitigation steps for now.
2021-12-13 34735, 2021
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.
2021-12-13 34722, 2021
lucifer
also solr 7 is unmaintained so it won't receive an update. solr 8 will
2021-12-13 34750, 2021
PopperBruda has left the channel
2021-12-13 34753, 2021
lucifer
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.
2021-12-13 34733, 2021
Freso
reosarevok: Go!
2021-12-13 34741, 2021
reosarevok
Hi!
2021-12-13 34758, 2021
zas
bitmap, reosarevok : IP blocked
2021-12-13 34722, 2021
reosarevok
I worked on updating open PRs to the latest master and fixing eslint / Perl::Critic issues found by new rules
2021-12-13 34741, 2021
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)]
