#metabrainz

I get the feeling that's the point

I'm guessing they're not going to like this tiny tiny window on the bottom left https://usercontent.irccloud-cdn.com/file/eQTRx...

monkey: https://developers.google.com/youtube/terms/req...

"Embedded players must have a viewport that is at least 200px by 200px. "

Yep, just reading that

So. Yeeeaaah.

how about we rip off the band aid in on go and remove YT support?

and just move on. fuck this noise.

not sure what to replace it with, but this silo BS is clearly not going to work.

mayhem: speed up funkwhale support :))

My vote would go to leaving it as-is for now

It's still useful even without a more appropriate quota

tandy1000: that doesn't solve the problem that YT solved for us.

you think?

maybe youre right

yes, the idea was the BP becomes usable by anyone.

FW is a very serious installation hurdle, which doesn't put the two solutions into the same category.

^ regardless of being logged in or having an account

funkwhale is probably not that widely used to have as big a library as yt, but also i realise people will need to configure to a server..

I think we just need to focus on more music service integrations.

BYO music service.

what about a toggle that enables YT and just makes the window big

I don't like Spotify so it's all would have used, but I also won't cry if it's removed. Doubt anyone's gotten used to it

monkey: uh no, i think we may have to stop using youtube at all. because youtube says the current thing is in violation of ToS.

Gah

That's a good idea aerozol.

i.e. acc to my understanding, this is a separate review before ratelimit quota consideration. even if we chnage all this, there is no guarantee that the qouta will be increased.

and even if we get it, then YT will arbitrarily change their rules, and shut us down for violating their rules.

we'll forever be beholden to them.

rip off the band aid. and 🖕 to you tube.

yeah true. that's what they did in 2020 when they removed the free search api.

and they'll do it again if they need to earn more money.

they've made adverts ridiculously intrusive now, purely to try and push youtube TV or whatever it's called

zas: hi! can we add localhost to allowed origins as well?

*allowed origins on staticbrainz

lucifer: why?

I am trying to test using static hosted fonts on staticbrainz.org, and my localhost setup gives CORS errors.

mayhem, monkey: beta.listenbrainz.org is now using fonts from staticbrainz.org

noice

Noice indeed

I've just updated test.LB again with latest (and most likely final for this PR) BrainzPlayer redesign changes

zas: if not on all resources, adding cors header for localhost on resources in `/fonts/` would be fine as well.

[listenbrainz-server] 14amCap1712 opened pull request #1881 (03master…fonts): Use self-hosted fonts from staticbrainz.org https://github.com/metabrainz/listenbrainz-serv...

once we are satisfied with LB stuff, I'll open tickets in other projects and point to this PR.

lucifer: test now

zas: not working yet. LB by default actually runs on port 7000. could that be an issue?

likely

what's the full origin url?

try again

still the same CORS error, zas

can you check headers?

(in server response)

try again

yup works now.

thanks zas!

I added a one-day cache for files in /fonts/

makes sense

it will be better to have a version on those so we could have much longer cache

in fact, most resources served by staticbrainz have such version

there is actually a version on those. https://staticbrainz.org/fonts/sintony-v11-lati...

v11

ah good

once you're done with experiments, we'll have to remove this specific header

the CORS one?

yes, it's specific to your port

because removing that'll break font rendering in development. i was hoping we could keep it permanently?

does it make sense to host these assets locally for development? (I don't even run on localhost, so it probably doesn't make sense to have this exception for all possible values...)

we could instead allow * but for specific IPs, but to me this setup (http + localhost + port) is very weird on public server

aerozol: This is what making the Youtube video visible by default could look like. It is a bit in the way but perhaps we can add a button to manually make it smaller (not sure that would fit YT's requirements though) https://usercontent.irccloud-cdn.com/file/e64bB...

ok i see.

not the wurst thing ever....

alastairp, i haven't seen that in use i.e. can we even add multiple url for same font format in css?

lucifer: oh, I hadn't considered that. do you mean an absolute url in a url() declaration in a css file?

yes

we can do multiple urls, 1 for format but not sure about multiple urls per format. will try and see

maybe less has some kind of environment-based variables that might help? It does mean hosting all the font files in each repo though

or maybe SB for prod and google fonts for dev?

yeah that could work.

is this only for these items in the .less file, or are there other web-hosted js or css files too?

for now, i think its only those items. but the intent is in future to host more stuff on SB.

just taking a step back - browsers now won't re-use items from the cache cross domain (a change that happened within the last year or so), so hosting something on SB and using it on MB and LB won't cause it to use a cached version

however there are other reasons to do so - e.g. to make the browser open more connections, some to MB and some to SB

afair the current motivation is GDPR.

today its gfonts, tommorrow cdns etc so better to move what can be hosted on staticbrainz there and mention rest in privacy policy etc.

sure, so that's just a question of getting them off of google fonts and on to somewhere that we maintain

ah you mean moving fonts under lb.org/ and mb.org/ ?

yes, right

but we'll need cors still for allowing it on localhost

the same cors rule just for lb.org/fonts instead of sb.org/fonts

I can't remember if the motivation for staticbrainz was to have a location where the same assets can go to all MeB projects and be cached once (in which case, this no longer works), or if it was to move assets to a separate domain to enable more connections from the browser (in which case this is still a good idea)

right, but if we hosted fonts from listenbrainz.org for production, we could host them from localhost for local dev

ah ok, the static manifest magic?

yes, maybe with the manifest, or with variables in less, or with relative paths in the css file (instead of absolute)

makes sense

atj: ping

zas: sorry, was just finishing lunch

do you have remote console access to the new server?

np, I installed a base system (ubuntu 20.04 hwe minimal) on aphex, can you check you have root access to it using your key

ok, one sec

we don't have remote console (that's on demand), but we have a rescue system (network-boot), that allows us to install the initial system

alright, just wanted to check the options if something bad happens ;)

we can start over from scratch, I'll explain details if we need that, but basically we have a web interface, we check rescue system, and reset the machine, it boots on rescue, on which we can login with provided temporary password

OK, root login works

so from that, we should be able to use ansible

I suggest we document steps in README.md in the repo you shared

yep, sounds sensible. I just ran bootstrap.yml and it looked good until it bailed out due to an empty netplan configuration

we need to be careful about network

we can easily lose connectivity ;)

indeed, that's my main concern, followed by the firewall

first, this machine has 2 NICs

default names aren't constant

(not the same on all machines)

that's the first issue

so we need a way to indicate which one is external NIC, which one is internal NIC

[metabrainz-logos] 14MonkeyDo merged pull request #14 (03master…master): MEB-49: added mid-detail version of MessyBrainz logo + bonus logo fix https://github.com/metabrainz/metabrainz-logos/...

the other is not configured at all at this stage

we use 10.2.2.x internally

x is the last number of the external IP, here 10.2.2.58

yeah I see. I think it may be safest to set the interface names in config variables for each host

yup

enp9s0 is the external interface

do you think using netplan to rename the interfaces to ext/int or something is stupid?

nope, that's not stupid, because it would simplify many things

I'm just thinking it would be nice to have consistent names on all the boxes eventually

on older servers, we use eth0 for ext, and eth1 for int

the systemd names are annoying

hard to type and remember

agreed

alright, give me 5 mins and I'll work add some configuration

*work on adding

we can then have a list of variables that are required for a successful bootstrap

well, external interface is required for sure

internal one too, physical for now, virtual later

I'm a bit nervous about this netplan configuration :)

we can reset if needed, that's the whole point of this

OK, I'm going to just use set-name on the internal interface for now, just read some conflicting reports about it

please commit changes you made to repo, so I can see

[listenbrainz-server] 14amCap1712 merged pull request #1879 (03master…fix-frontend-message): Add message to inform user of listen deletion policy https://github.com/metabrainz/listenbrainz-serv...

(even if not definitive)

release time!

monkey: any other PRs to merge?

also, akshaaatt any PRs ready to merge?

Redesign brainzplayer PR looks ready to merge from me lucifer once monkey has looked at it