Conversation

Notices

1. Embed this notice
   NEETzsche (neetzsche@iddqd.social)'s status on Friday, 10-Nov-2023 22:38:47 JST NEETzsche

   Displaying the latest scrobble now in BalormoFE: https://git.pleroma.social/pleroma/pleroma-fe/-/merge_requests/1865

   Thanks @hj

   • Embed this notice
     NEETzsche (neetzsche@iddqd.social)'s status on Saturday, 11-Nov-2023 11:39:51 JST NEETzsche

     in reply to

     • :archlinux:

     I wrote a little bash script that moves Last.fm scrobbles to BalormoBE. I run a cron job. You don’t necessarily need to use my script but it should at least give you an idea.

     https://github.com/TheStranjer/fediFM

   • Embed this notice
     :archlinux: (lewdthewides@hidamari.apartments)'s status on Saturday, 11-Nov-2023 11:39:52 JST :archlinux:

     in reply to
     @NEETzsche @hj I think I'll spend all night figuring out how to get this working
   • Embed this notice
     :archlinux: (lewdthewides@hidamari.apartments)'s status on Saturday, 11-Nov-2023 12:52:14 JST :archlinux:

     in reply to
     @NEETzsche @hj I saw that when I found your soapbox MR
     
     Seems like a tall order to get pleroma scrobbling support into all the clients. Is it even possible to get the endpoint working with client that support ListenBrainz or Maloja?
     NEETzsche likes this.
      repeated this.
   • Embed this notice
     NEETzsche (neetzsche@iddqd.social)'s status on Saturday, 11-Nov-2023 12:58:14 JST NEETzsche

     in reply to

     • :archlinux:

     I think adding ListenBrainz and/or Maloja support would be phenomenal, but for me it’s one step at a time. Right now I have an open PR into PleromaBE to make it so that the account/pleroma section in the AccountView JSON has a scrobbles entry, which is an array of the most recent five scrobbles, if any. You can see that PR here: https://git.pleroma.social/pleroma/pleroma/-/merge_requests/3975

     I’m doing this because as it stands, my implementation in the FE requires hitting the endpoint, and in order to make it avoid absolutely inundating it, I have to throttle it. This would obviate that completely and allow me to just look at the returned account data normally, as with most of the rest of the API.

     So far Lanodan seems reluctant to add it, saying that Listen is in itself a “hack” that first (he stresses that word) must be redone. Basically, the complaints about scrobbling are that they aren’t supported by FEs, but then efforts to support it get rebuffed.

      likes this.
   • Embed this notice
     NEETzsche (neetzsche@iddqd.social)'s status on Saturday, 11-Nov-2023 13:48:09 JST NEETzsche

     in reply to

     • :archlinux:

     So, this is the person “reviewing” my code. It’s not going to be accepted, not because of the content of it, but because of who I am. Any basis for rejection will be entirely pretextual at best.

   • Embed this notice
     NEETzsche (neetzsche@iddqd.social)'s status on Saturday, 11-Nov-2023 14:20:30 JST NEETzsche

     in reply to

     • :archlinux:
     • Christi Junior
     • LukeAlmighty

     @LukeAlmighty @ChristiJunior @CumskinFoidPuncher69420 I just wanted to make free software

     LukeAlmighty likes this.
   • Embed this notice
     The Problem :verified_pink: (marine@breastmilk.club)'s status on Saturday, 11-Nov-2023 14:25:07 JST The Problem :verified_pink:

     in reply to

     • :archlinux:
     • Haelwenn /элвэн/ :triskell:
     • Christi Junior
     • LukeAlmighty

     @NEETzsche @ChristiJunior @LukeAlmighty @lewdthewides @hj

     so basically we’re not getting scrobbling because @lanodan doesn’t like you because you’re you which is disappointing at a baseline. Why are we removing features when the point is to be adding them. If we’re worried about ‘security risks’, at that point, none of us should even be allowed to post anything but text since any attachment can also attack security vulnerabilities (ie; the way that Claire from Chudbuds got hacked via a minecraft mod pack).

     NEETzsche likes this.
   • Embed this notice
     NEETzsche (neetzsche@iddqd.social)'s status on Saturday, 11-Nov-2023 14:26:45 JST NEETzsche

     in reply to

     • :archlinux:
     • Haelwenn /элвэн/ :triskell:
     • Christi Junior
     • LukeAlmighty
     • The Problem :verified_pink:

     They didn’t even mention “security risks,” assuming there even are any. There are no “security risks” to delivering a little bit of JSON from a scrobbles table that only has text in it, and there definitely aren’t any new “security risks” considering this exact content can be retrieved from another endpoint.

   • Embed this notice
     NEETzsche (neetzsche@iddqd.social)'s status on Saturday, 11-Nov-2023 14:32:05 JST NEETzsche

     in reply to

     • :archlinux:
     • Haelwenn /элвэн/ :triskell:
     • Christi Junior
     • LukeAlmighty
     • The Problem :verified_pink:

     Here’s the goalpost for technical basis to reject this PR. It must meet ALL of the requirements:

     • It must introduce a bug that wasn’t already there
     • It can’t be from data errors created by another endpoint
     • The same complaint can’t be issued against any other endpoints already in the codebase, example: if the complaint is that you can inundate the scrobbles endpoint, then the same complaint can be made about the statuses endpoint, making this an invalid basis for rejection
     • It must reproducible

     I’m not setting this goalpost because I think it will be met, I’m setting it because I want to give those who aren’t technical a baseline for evaluation here. If ANY of these requirements aren’t met, we can conclude that it’s not about any technical issue. It’s about who made the PR.

      likes this.
   • Embed this notice
     NEETzsche (neetzsche@iddqd.social)'s status on Saturday, 11-Nov-2023 14:52:30 JST NEETzsche

     in reply to

     • Sheriff CJ (The Impostor)??
     • The Problem :verified_pink:

     Scrobbling is literally a function

   • Embed this notice
     Sheriff CJ (The Impostor)?? (colonelj@freespeechextremist.com)'s status on Saturday, 11-Nov-2023 14:52:31 JST Sheriff CJ (The Impostor)??

     in reply to

     • The Problem :verified_pink:
     @marine @NEETzsche @hj form over function
   • Embed this notice
     The Problem :verified_pink: (marine@breastmilk.club)'s status on Saturday, 11-Nov-2023 14:52:32 JST The Problem :verified_pink:

     in reply to

     • Sheriff CJ (The Impostor)??

     @colonelj @NEETzsche @hj sometimes your opinions are unnecessary, but we still have to see them and hear about them.

   • Embed this notice
     Sheriff CJ (The Impostor)?? (colonelj@freespeechextremist.com)'s status on Saturday, 11-Nov-2023 14:52:33 JST Sheriff CJ (The Impostor)??

     in reply to
     @NEETzsche @hj seems unnecessary.