#[schmarty]barnaby++ woo, thanks! migrated. i'll see if any tokens broke for 0.1.0 tomorrow when i post caturday.
#Loqibarnaby has 16 karma in this channel over the last year (29 in all channels)
#[schmarty]had to polyfill `str_contains` because I have PHP7.4 on the box where I run all this 😅
#[schmarty]that clears up my main concern w/ updating taproot\indieauth to 0.2.0, which is exciting. i can also undo my `client_id` wrapping now that unfetchable `client_id` is no longer fatal.
smudge-the-cat and oodani joined the channel; smudge-the-cat left the channel
#barnaby[schmarty] oops sorry about the str_contains thing, I had no idea that was a PHP 8 only feature! I’ll replace that with something compatible with earlier versions, then release version 0.2.1
#barnabygood catch, thanks for testing! [schmarty]++
#Loqi[schmarty] has 15 karma in this channel over the last year (32 in all channels)
[KevinMarks], tetov-irc and jjuran joined the channel
#[schmarty]Ehehe an unexpected upside to my foot dragging
geoffo joined the channel
#barnabyif I had written tests which actually ran the script rather than just testing the migration function, it would have been picked up in the CI, but that seemed like too much effort for a tiny script. Perhaps I should get that codesniffer we use for php-mf2 working to check stuff like that statically
geoffo joined the channel
#[tantek]4Pretty sure CASSIS still only depends on PHP5+ functions
#GWGI do a polyfill on str_contains because it's really useful
#IWDiscordGateway<corlaez> Regarding h-entry/h-feed microformat. Has it been considered to assume the u-url to be the link rel="cannonical" if it is missing?
#IWDiscordGateway<corlaez> I am capable of generating my own u-url but for some pages I will like to not show it (not even as a hidden link because a anchors with href aren't supposed to be hidden.
#IWDiscordGateway<corlaez> And with all my pages having a cannonical I was wondering if the u-url could be assumed to be the cannonical if present.
#IWDiscordGateway<corlaez> If there is no u-url and no cannonical I guess we could fail the validation
#IWDiscordGateway<corlaez> Regarding h-entry/h-feed microformat. Has it been considered to assume the u-url to be the link rel="cannonical" if it is missing?
#IWDiscordGateway<corlaez> I am capable of generating my own u-url but for some pages I will like to not show it (not even as a hidden link because a tags with href aren't supposed to be hidden)
#IWDiscordGateway<corlaez> And with all my pages having a cannonical I was wondering if the u-url could be assumed to be the cannonical if present.
#IWDiscordGateway<corlaez> If there is no u-url and no cannonical I guess we could fail the validation
#IWDiscordGateway<corlaez> (not even as a hidden link because a tags with href aren't supposed to be hidden)*
#aaronpkone problem with that is when you have multiple h-entrys on a single page, any kind of list of posts
#[snarfed]1also we can easily think of examples where the u-url isn't actually canonical
#aaronpkoh yeah, i was assuming you meant canonical within the context of the site, but not actually what rel=canonical means
#IWDiscordGateway<corlaez> ah ok, I am not into POSSE that much but I see how taking that into consideration makes u-ul a requirement. Ok thanks!
#IWDiscordGateway<corlaez> and multiple h-entrys make sense too. I mean if I was publishing them together it may be my responsibility but if someone else is aggregating them the lack of u-url is problematic.
[marksuth], jacky and gxt joined the channel
#[tantek]4not sure if this is worth adding to our /privacy page but perhaps for consideration (interesting chart for those that run websites that may retain information from others, e.g. caching data from webmentions) https://twitter.com/jtrevorhughes/status/1571952116733825025
#[schmarty]barnaby: seems like the migration script worked fine btw. i've just updated to 0.2.0 and i'll let you know if anything blows up when i next post (which is: whenever)
#[schmarty]one thing that is still a bit of a struggle is Owncast's indieauth client integration which provides a path-less URL and taproot still rejects this. give it a shot if you have a test site set up. it's authentication-only: https://watch.owncast.online/
#barnaby[schmarty] thanks! I’ll release a new version with the script in then
#barnabythe client_id and redirect_uri comparisons are an important part of the spec so I don’t want to make changes to them until it’s at least unofficially official
#barnabyand ideally I’d wait until the spec is updated, then change the library based on that
jacky and gRegor joined the channel
#gRegoraaronpk, did you ever hear back on that OAuth vulnerability article from Detectify?
#aaronpkoh shoot, that was on me to send him some corrections, which I promptly forgot about
#[schmarty]thanks barnaby. i left my unsolicited 0.02¢ in the PR. i believe it's important to always be consistent but don't particularly think there's a risk from storing/comparing the normalized version only.
#[schmarty]that said, i also think it's fine to only compare and store the non-normalized version and treat it as an opaque string until it is to be "used" either to validate it or fetch it and only normalize it at those times.
#barnabythat’s probably the case, but it also sounds like exactly the kind of thing which accidentally causes some obscure vulnerability, so I wanted to get some additional opinions about it
#barnabyyeah lazy normalization could be a good approach too
#barnabyit wouldn’t introduce too much additional complexity either, which I definitely want to avoid when it’s not mandated by the spec
tetov-irc, jacky, geoffo and neceve joined the channel