2023-04-09 UTC
RasAlGhoul, bob1, RasAlGhoul_, Retro, antranigv, fef and [pfefferle] joined the channel
# 12:51 [pfefferle] the webmention plugin shows the wrong URL for brid.gy for example
fef joined the channel
# 14:02 GWG I saw that...but I lacked context for you mentioning it now.
# 14:14 [pfefferle] and I get an 500 error if I try to use the internal parser 😞 `/webmention/1.0/parse/`
# 14:15 GWG I intend to work on some of these
# 14:24 GWG I'm looking, and some of the bridgy webmentions on my site didn't migrate to other types.
# 14:26 GWG Also, since the links are down, I can't try reparsing. May have to put in something to allow people to fix.
# 14:36 GWG I actually pushed that into the repo, but forgot to release a new version.
# 14:36 GWG It was a S instead of s somewhere
IWDiscord, Gremblo[d], Silicon[d], steinaech[d], Tuck[d], shaunix[d], marksuth[d], Hazel_[d], Bainstor[d], singlequbit[d], Grayson[d], indieweb-irc-bri, FoondyChop[d], DJ_[dj_je][d], IndieWebCamp_IRC, who[d], Nezteb[d], fernandolguevara, hollie[d] and aaronpk[d] joined the channel
# 15:08 [pfefferle] now we have A LOT of broken URLs because of abandoned brid.gy URLs
# 15:08 [snarfed] I can look into serving them from storage instead of the API
# 15:10 [pfefferle] but I will also have a look, to use the permalink of the tweet in the meantime
# 15:10 [pfefferle] we do not store it, because it would mess up the db with a lot of unused data
# 15:10 GWG [pfefferle]: We were supposed to move the author url into the Author url field
# 15:11 GWG [snarfed]: Can you keep them up from storage for a bit at least?
# 15:12 [pfefferle] I will re-think the db design and make a doc, to not do this again!
# 15:13 GWG [pfefferle]: Document it and we'll figure out a solution
# 15:19 [snarfed] disk is cheap, databases are good at storing lots of data, in general I lean toward storing data in case I need it in the future
# 15:19 [snarfed] I'll look at serving Bridgy Twitter URLs from storage now
# 15:20 [pfefferle] I can not find a point where we check if a new webmention is an update or not 😳
# 15:23 GWG It uses author url as a fallback only
# 15:26 [pfefferle] it includes the source at this point, but it we should use `$commentdata['comment_meta']['webmention_source_url']` or `$commentdata['source']` instead
# 15:27 GWG I think I should push off [chrisbergr]'s PR until we work on some of the other issues, then come back to it. I've been looking at that problem,. but better to work on fixes for people's issues before enhancements
# 15:30 GWG [chrisbergr]: I want to get a version into the next major release, but I think this takes precedence
RasAlGhoul_ joined the channel
# 15:48 [snarfed] ok [pfefferle] GWG, Bridgy Twitter URLs are now serving from storage
# 15:48 [snarfed] (only from the last 18 months or so, since I don't keep them in storage forever)
# 15:53 Loqi [snarfed] has 3 karma in this channel over the last year (135 in all channels)
# 15:55 GWG [pfefferle]: Suggest we look for things that aren't needed anymore and for places we can add comments for future versions of ourselves
# 16:00 GWG [snarfed]: And I thought it would be smooth sailing post 5.0
# 16:02 [snarfed] hah, no way, there are always bugs and issues to fix with a change this big
# 16:02 [snarfed] and they often show up slowly after a while, not just immediately
# 16:02 [snarfed] GWG++ pfefferle++ again, thank you for landing the merge and fixing these issues!
# 16:02 Loqi GWG has 17 karma in this channel over the last year (60 in all channels)
# 16:04 GWG I thought it would be about adding in features like avatar caching and finer control of the templates... maybe parsing reviews...
# 16:04 GWG I didn't think there would be no bugs, but...
# 16:09 GWG [snarfed]: Do the stored Twitter Bridgy pages have non broken avatar URLs in them?
# 16:10 GWG I am still wondering if I could add in the code to store them before they vanish.
# 16:11 GWG I know [pfefferle] wants to use avatar privacy, but it doesn't store Webmention avatars, only gravatars.
# 16:12 GWG I think the Author of that plugin is willing to work with us though
# 16:12 GWG We just have a solution ready to go now.
# 16:13 [snarfed] GWG Bridgy Twitter's avatar URLs are Twitter's own URLs. Bridgy doesn't store or re-serve images
# 16:14 [snarfed] so yes they generally work, but not all, eg I think if someone changed their profile picture, URLs to their old profile pictures may not work
# 16:14 GWG [snarfed]: I know.. just wondering how many are broken
# 16:15 Loqi [preview] [snarfed] #269 Backfill avatars on old comments
# 16:16 GWG [snarfed]: Maybe I should do a new PR that Avatar Privacy could trigger or I could use the local code we wrote for same..
# 16:17 GWG But the trigger for refreshing old avatars can be on the Webmention sode
# 16:18 [snarfed] I'd recommend putting it in a WP-CLI command or standalone script. for big sites, doing it in background cron seems like a recipe for pain
# 16:19 [snarfed] eg I have ~15k comments on my site, and it's probably only small to medium, I expect other people have way more
# 16:19 GWG [snarfed]: We support multiple ways for migration now
# 16:20 GWG No, we just had time to discuss and rediscuss
# 16:20 GWG But that's why I have moved to advocating for simple structures easy to hang different methodologies on
# 16:20 GWG So we can build a nice default and let others enhance it if they want
# 16:21 GWG So... opinions with ways to make options
# 16:21 GWG Although some things have to be as settings
# 16:31 Loqi GWG has 18 karma in this channel over the last year (61 in all channels)
# 16:31 Loqi [pfefferle] has 16 karma in this channel over the last year (30 in all channels)
RasAlGhoul and RasAlGhoul_ joined the channel
# 21:31 [chrisbergr] Is avatar caching really a feature that belongs in the webmention plugin instead of its own?
# 21:36 [snarfed] discussed here on Fri, [pfefferle] agrees, the Avatar Privacy plugin seems like a good existing one, not sure if it fully works yet with webmention avatars though?
# 21:40 GWG I'm happy to go my own way if majority feels differently. Thus the statement about making it flexible
# 21:42 GWG I tried turning it in and all my avatars vanished
# 21:44 GWG At least, it doesn't work retroactively
# 21:46 GWG Where on your site is it storing a Webmention avatar as opposed to a gravatar?
# 21:47 [snarfed] specifically, If I understand him right, he said it works with existing webmention avatars, just on demand. it doesn't proactively go fetch and store them as soon as you install it
# 21:47 [pfefferle] Ah, now I get it... no, you can't restore images if you haven't used it before
# 21:47 GWG It is hard to tell what is a Webmention on notiz.blog. Can you show me one that is being stored?
# 21:48 Loqi [preview] Vor zwei Jahren wollte Twitter in das „Dezentrale Netzwerke“-Business einsteigen und gründete eigens dafür das Projekt Bluesky. In den folgenden zwei Jahren wurde viel evaluiert und diskutiert, was wohl die beste Lösung für Twitter sei und wi...
# 21:50 [pfefferle] (a second raison why I would love to use a separate plugin instead of a webmention one)
# 21:54 GWG [pfefferle]: I am looking at the code
# 21:57 Loqi [pfefferle] has 17 karma in this channel over the last year (31 in all channels)
# 21:58 GWG [pfefferle]: If we want to trigger it retroactively... I have to figure out how it works
# 22:04 GWG [pfefferle]: I would at least have to address retroactive. I'm looking for how to do that.
# 22:07 [snarfed] GWG on that issue, the Avatar Privacy author said to just use an existing cache warming plugin, or write a script to do it by fetching all posts
# 22:08 GWG I still want to know how it saves it
# 22:11 GWG A cache warming plugin would reload every post to address the issue, which I suppose is fine
# 22:15 Loqi [preview] [David Shanske] Why don’t people wear hats outdoors anymore in large numbers? In the past, it was considered inappropriate to go bareheaded in some contexts.
# 22:16 GWG Or specifically, Webmentions with gravatars
# 22:17 [snarfed] Oh the avatar privacy plugin? 🤷♂️ I haven't actually tried it
# 22:18 Loqi [preview] [dshanske] #270 Does Not Cache Gravatars received via Webmention
# 22:18 GWG That's why when I tried it...the last Webmention avatars disappeared...they used Gravatar
# 22:22 GWG I also need to see the cache time
# 22:23 [snarfed] I don't follow though. my h-card profile picture (which is therefore the one in the webmention on your site) isn't a gravatar
# 22:23 GWG Hmm...it only stores it for a day or t6
# 22:28 GWG [snarfed]: But we have an issue with avatar images no longer being available... which I suppose is fine because it would now replace it with a mystery image... better than a blank space, I suppose
# 22:28 GWG I'm just thinking how s bunch of silhouettes will look
# 22:30 [snarfed] Right. They're unavailable *_now_.* This might change them, but it wouldn't make them worse
# 22:31 GWG This is why I'm tempted to run a parallel solution that stores them forever.
# 22:31 [snarfed] And individual sites would opt into avatar privacy themselves. you all wouldn't be imposing this on anyone, you'd just help make it compatible
# 22:32 [snarfed] Eh, this us the same conversation you had with [pfefferle] earlier
# 22:32 GWG [snarfed]: I don't see an option to turn off the cache invalidation.
# 22:33 GWG I intend to work with the developer, but I still might not choose it for myself
# 22:33 [snarfed] There are so many other webmention plugin bugs and improvements to be done, I don't see why you'd prioritize reinventing the wheel here, but up to you all
# 22:34 GWG [snarfed]: I'm not prioritizing it, we already wrote it.. I'd just take the PR and put it into a separate file.
# 22:34 GWG And remove the Gravatar caching code from the plugin entirely
[tantek] joined the channel
# 22:37 [snarfed] The initial implementation is only the first 20% of the work; the next 80% is maintaining it 😎
# 22:37 GWG It just means we'd remove the Gravatar and the local mystery silhouette from the code and let avatar privacy or something handle it
# 22:37 GWG I know... my intention is to remove the Gravatar code for now