#dev 2018-08-23

2018-08-23 UTC
ben_thatmustbeme, KevinMarks_, jeremych- and [iambismark] joined the channel
#
[iambismark] Does MF2-parser-as-a-service exist? No need for integrating libraries, still just one HTTP request, etc...
#
tantek__ iambismark good point. I believe many such exist at microformats.io
Kazuto, tantek__ and snarfed joined the channel
#
tantek__ aaronpk, the other approach is what we did with standardizing Webmention in W3C
#
tantek__ we had loose coupling to mf2, with the option for folks to use things other than HTML etc.
#
tantek__ this allowed a "market-testing" of which formats for publishing / processing webmentions would gain the most traction, instead of a W3C committee deciding a priori
#
tantek__ now that over a year later it's pretty clear mf2 is the way webmentions work in practice (no one has followed up JSON(-LD) attempts), it would be reasonable to document the industry practice in a Webmention 1.1 which makes mf2 processing explicitly a SHOULD, while still specifying how non-HTML methods can work with possibly a note saying that in practice, early experiments with that have not had any market success.
#
tantek__ I realize you're trying to lock down more things with IndieAuth (given the history of vagueness / lack of interop with OAuth), however, IndieAuth could likely benefit from another such "loose coupling" to WAM/h-app etc. to allow publishers / consuming code to work it out in the market
#
aaronpk That could be the way to go for this first draft
#
tantek__ one thing we can do here is make sure the different syntaxes whether h-app or WAM JSON match up semantically
#
tantek__ so that there's a defined translation from one to the other
#
aaronpk although I suspect people might then suggest using the openid connect authorization server metadata document as the way to send that metadata
#
tantek__ that way you can specify IndieAuth based on that abstraction, regardless of where the app info comes from
#
tantek__ sure people can suggest that - but proprietary internet draft with zillions of properties is much worse than a W3C spec already being implemented in browsers
#
tantek__ than a *simpler* W3C spec at that
#
tantek__ and you can point out that 3 of 4 major browser vendors are active participants (editors!) of the Web App Manifest spec, so it's happening
#
tantek__ https://w3c.github.io/manifest/
#
tantek__ and Intel too
dmcweeney joined the channel
#
dmcweeney Can anyone help me as to what the CSS code would be for removing space after the footer at the bottom of my website?
#
@jgmac1106 @pauljacobson here is link to my original post: https://jgregorymcverry.com/update-on-badging-with-webmentions/ Been experimenting with simplified badges using webmentions. Getting close. (http://jgregorymcverry.com/6230-2/) (twitter.com/_/status/1032432201336594434)
#
dmcweeney When you go to drewmcweeney.com and scroll down you will see space after the footer. Before I even switched hosts to reclaim hosting, there was no space after the footer
#
[iambismark] dmcweeney: it appears to be caused by this tag: `<img alt="css.php" src="http://drewmcweeney.com/wp-content/plugins/cookies-for-comments/css.php?k=55f45030a624519681dde3c75bdbf782&amp;o=i&amp;t=2141037806" width="1" height="1" scale="0">`
#
dmcweeney Oh-What should I do?
#
[iambismark] looks like it may be a wordpress plugin inserting this? thats a bit outside my wheelhouse, maybe someone with more WP knowledge than me can help
#
sknebel that's an antispam plugin reclaim installs by default, you can turn it off or maybe configure it to use something else instead
eli_oat joined the channel
#
[iambismark] is it possible to update your theme so that the tag is inside the `<div id=wrapper>...</div>`?
#
dmcweeney Is it the cookies for comments plugin or antispam plugin that I should deactivate?
#
[iambismark] appears to be “cookies-for-comments”
renem joined the channel
#
dmcweeney It worked! I deleted it! Thank you so much!! I am so glad I am able to ask around to experts like you guys!! I'm a beginner learning how to do website so it is nice to know how to figure out things once I ask! Thanks!!
miklb, myfreeweb[m], strugee, tantek, [eddie], [tantek], [shaners], tantek__, [aaronpk] and Atlas29 joined the channel
#
Atlas29 | .- _ '-.
#
Atlas29 | / (o) _ _\
barpthewire and cweiske joined the channel
#
@ndw Thoughts on implementing webmention. Or not. https://so.nwalsh.com/2018/08/23/webmention (twitter.com/_/status/1032515870600425472)
barpthewire1, barpthewire, Zegnat, KevinMarks and Matrixiumn joined the channel
#
Matrixiumn | _.--"""""--._
jeremycherfas and [kevinmarks] joined the channel
#
[kevinmarks] This is interesting re mf2py performance - is out datetime parsing like this? https://untar.hackl.es/burned-marshmallows
#
sknebel [kevinmarks]: not really. for VCP it has to do a bit of somewhat complex regex (which potentially still has room for tuning, I didn't really touch that area), but no complex guessing what's what logic, and normal dt properties the contents aren't parsed at all
#
Zegnat Someone say regex?
#
Loqi Zegnat: sknebel left you a message 10 hours, 57 minutes ago: gRegor​Love says "Zegnat: I added h-app and rel-icon to indiebookclub"
KevinMarks and [jgmac1106] joined the channel
#
[jgmac1106] @dmcweeney yes deactivate cookies for comments, and then activate askimet, it is an antispam plugin your webhost includes on all fresh installs. There is a 1x1px image in the cookie that messes up stuff
#
sknebel it's also supposed to have as setting for a css-based mode, that should be fine
[kevinmarks], Cbcj, barpthewire and arlen joined the channel
#
arlen | -- --
#
arlen | /:" | /:'" |
#
arlen | -- ___ ___ -
[pfefferle], [jgmac1106] and deetwelve3 joined the channel
#
deetwelve3 .. _ .. ""..
barpthewire, [pfefferle], [metbril], TripFandango and [scottgruber] joined the channel
#
[scottgruber] Morning! I’m trying to send Webmentions from my site and am wondering if anyone could help test my server or let me know the curl command? I’m receiving them now and would like to send to twitter and mastodon silos using Bridgy and Bridgy fed. If I get it all working with Perch Runway, I’ll update the wiki. TIA.
#
sknebel what kind of help do you need?
#
sknebel you can check if the basics work e.g. through webmention.rocks
#
[scottgruber] Do I confirm with a curl command and if so what is the syntax? I can receive a webmention now, but sending failed during tests last night. I’m chcking webmention.rocks now.
#
Zegnat [scottgruber], webmention.rocks gives you some links you can hang over to your CMS to see if it correctly sends webmentions to them.
#
Zegnat https://indieweb.org/Webmention-developer#How_to_send_webmentions_with_cURL is the closest we have to a curl command, though I am not sure what you are able to test with that.
dansup joined the channel
#
[scottgruber] Sorry, its my ignorance talking. 😀
#
[scottgruber] I’m on the webmentions.rocks site and following the instructions to test. brb.
leg, dansup, KevinMarks_, jackjamieson, gRegorLove, KevinMarks, KartikPrabhu, barpthewire, snarfed, [shaners], [pfefferle], [jgmac1106], jgmac1106_, tantek__, [dougbeal] and tantek joined the channel; dansup left the channel
#
GWG Zegnat, client information discovery is merged into IndieAuth for WordPress. After it is released to everyone, more people need manifests
#
Zegnat GWG have you documented the properties you use / fallback on somewhere?
#
GWG I can later. Where is good?
#
GWG I built first
#
snarfed rough consensus and running code
#
GWG I built simply
#
GWG snarfed?
#
snarfed ietf's motto
#
GWG I am missing context
#
snarfed lmgtf: https://ietf.org/how/runningcode/
#
GWG Oh, and sorry to leave my PR broken.
#
snarfed np! we don't need it unless it fixes miklb's problem
#
GWG I am missing something obvious there about why my check is failing
#
GWG This is why I hate define
#
snarfed ignore it until miklb tests with it
#
GWG I want to know why if MICROPUB_LOCAL_AUTH is true when it is defined as 0 regardless
#
GWG I feel like I am missing a ba
#
GWG sic PHP thing
#
GWG That's what I'm trying to figure out
#
Zegnat What is WAM?
#
Loqi Web App Manifest is a W3C living document that defines a JSON sidefile for web applications to publish information such as their name, icon, description https://indieweb.org/WAM
#
Zegnat Consumers and previous consumer details are on there, GWG
#
Zegnat If you could just add to it, that would be awesome!
#
GWG Zegnat, you do PHP, any thoughts on that?
#
Zegnat Link?
#
GWG https://github.com/snarfed/wordpress-micropub/pull/162/files#diff-4DA5A8AF3B72BADBB8E253C48BF9A16CR50
#
GWG Why is this returning true instead of false?
#
GWG I feel like I am missing something obvious
#
Zegnat What does apply_filters do?
#
GWG Allows you to override the return value. But I didn't set one.
#
GWG I can check it without the filter, I suppose
#
GWG But it wasn't working properly before for some people
#
Zegnat So you are really just passing the inverse result of class_exists() there
#
GWG Wondering if different versions of PHP evaluate 0 differently than '0'
#
GWG Yes, or the definition
#
Zegnat Sure, but you are setting that false
#
GWG I don't want the file to be loaded if it is true, but I do if it is false.
#
GWG So, if class exists or variable is true, don't load file
#
GWG Did I double negative?
Guest45420 joined the channel
#
Zegnat What is the use of MICROPUB_LOCAL_AUTH?
#
Loqi It looks like we don't have a page for "use of MICROPUB_LOCAL_AUTH" yet. Would you like to create it? (Or just say "use of MICROPUB_LOCAL_AUTH is ____", a sentence describing the term)
#
aaronpk I think you're right about the double negative
#
Zegnat Currently, as long as you set that constant to 0 (or '0', no difference), that part of $load is just tagging on “|| false”. Checking something for “or false” is useless, as it is the same as only checking the first part
#
aaronpk you're saying "load the file if disable_micropub_auth returns true" which is the opposite of what you just described
#
aaronpk you want "load the file if enable_micropub_auth returns true"
#
aaronpk your function is written to be "enable micropub auth"
#
Zegnat Woops, right, you probably shouldn’t have the ! on the return there
#
aaronpk also I don't understand what the 'disable_micropub_auth' filter is supposed to do either
#
Zegnat I was still trying to understand the reasoning for the constant and didn’t look at the return. Silly me.
#
GWG It turns off built in IndieAuth so IndieAuth the plugin can do it
#
GWG I find the name MICROPUB_LOCAL_AUTH one I regret for the same reason
#
aaronpk that's fine, just rephrase all this code in the positive
#
GWG Poor naming choice
#
aaronpk "MICROPUB_LOCAL_AUTH" is fine, that says "use the local micropub auth code" which is positive
#
GWG But it doesn't do that
#
aaronpk wait then what does it do
#
GWG It turns off using the built-in code
#
GWG In favor of another plugin
#
aaronpk oh right lol because the built-in micropub auth is actually remote auth
#
aaronpk jeez
#
GWG The built-in code uses Indieauth.com
#
aaronpk and changing that constant name is not an option?
#
GWG See why I said that I regret the naming choice
#
GWG It is
#
GWG In theory
#
GWG Somehow, without setting the constant, miklb was loading both files and that is why his install was failing
#
aaronpk then class_exists( 'IndieAuth_Plugin' ) must be returning true
#
GWG It is supposed to, if that plugin is active
#
aaronpk what does apply_filters do
#
GWG How about we look at the old version
#
aaronpk oh I know what's going on
#
GWG https://github.com/snarfed/wordpress-micropub/blob/master/micropub.php#L49
#
GWG You do?
#
GWG I am stumped
#
aaronpk maybe, checking old version
#
GWG That's why I asked and tried to just eliminate the thing entirely
#
aaronpk what determines which order the plugin files are loaded
#
aaronpk because on that line 49, if the indieauth plugin has not yet loaded, it will then load the micropub authorize file
#
GWG Nothing predictable.
#
aaronpk so you can't use the presence of a class to determine whether the plugin is there
#
GWG Good point
#
GWG I need something else.
#
GWG Hmm...
#
aaronpk can you look for the file on disk?
#
GWG Not predictably
#
GWG Let me think about that.
#
aaronpk don't you know the name of the folder it gets installed in?
#
aaronpk tho I guess that doesn't tell you if the plugin is activated
#
aaronpk does this work? https://codex.wordpress.org/Function_Reference/is_plugin_active
preaction17 joined the channel
#
preaction17 /\
#
GWG Hook is better. Have it run on init so I can dictate the order.
#
GWG We do that with other plugins
#
GWG Thanks, that helps.
#
tantek__ what is DTP
#
Loqi Data Transfer Project is a collaboration of organizations committed to building a common framework with open-source code that can connect any two online service providers, enabling a seamless, direct, user initiated portability of data between the two platforms https://indieweb.org/DTP
#
tantek__ DTP << 2018-08-15 [https://lwn.net/Articles/762601/ The Data Transfer Project]
#
Loqi ok, I added "2018-08-15 [https://lwn.net/Articles/762601/ The Data Transfer Project]" to the "See Also" section of /Data_Transfer_Project https://indieweb.org/wiki/index.php?diff=51301&oldid=50213
#
tantek__ what is a codebase
#
Loqi It looks like we don't have a page for "codebase" yet. Would you like to create it? (Or just say "codebase is ____", a sentence describing the term)
#
tantek__ what is a repository
#
Loqi A repository (AKA repo) is a collection of typically code & related versioned files in a folder (hierarchy), issues & their responses, and a set of (named) releases, often hosted in a source control backend such as Git, on a service like GitHub, or on independent sites using open source software https://indieweb.org/repository
#
tantek__ codebase is /repository
#
Loqi ok
#
loqi.me created /codebase (+23) "prompted by tantek__ and redirect added by tantek__" (view diff)
#
tantek__ codebase << 2018-08-21 [https://www.monkeyuser.com/2018/implementation/ Implementation] <blockquote>This is the codebase. It‘s amazing how such a small thing can do so much! [Dependencies]</blockquote>
#
Loqi ok, I added "2018-08-21 [https://www.monkeyuser.com/2018/implementation/ Implementation] <blockquote>This is the codebase. It‘s amazing how such a small thing can do so much! [Dependencies]</blockquote>" to the "See Also" section of /repository https://indieweb.org/wiki/index.php?diff=51303&oldid=49982
KartikPrabhu joined the channel
#
GWG Good talk
#
tantek__ ?
KevinMarks, KartikPrabhu and [kevinmarks] joined the channel
#
miklb ok, what is the easiest way to checkout GWG's PR to snarfed's micropub repo which I have checked out?
#
miklb https://github.com/snarfed/wordpress-micropub/pull/162
#
Loqi [dshanske] #162 Change config to filter
#
miklb is adding another remote the only option?
#
aaronpk you can go to https://github.com/dshanske/wordpress-micropub/tree/authfilter and download the zip
#
aaronpk if you want to avoid git stuff
#
miklb it's all a repo on my server so git might actually be easier than downloading a zip and swapping out installs.
#
aaronpk ah then yeah in that case I usually follow the instructions from github in the PR
#
aaronpk tho it looks like they don't show that to me because I don't have write access
#
miklb exactly
#
miklb if it's my repo, it's easy to get a PR. If it is someone else's not so much.
#
miklb so it seems
#
aaronpk git checkout -b dshanske/authfilter master; git pull https://github.com/dshanske/wordpress-micropub.git authfilter
#
aaronpk I think
#
miklb lol it helps if you are actually in the repo. this did work git fetch origin pull/162/head:authfilter
#
aaronpk whoa I haven't seen that before
#
miklb that creates a branch authfilter
#
aaronpk is that something github-specific built in to the git command?
[manton] joined the channel
#
miklb not sure, but I don't believe I have any scripts in my shell on my server that does fancy github stuff
#
miklb GWG that https://help.github.com/articles/checking-out-pull-requests-locally/
#
aaronpk wow cool
#
aaronpk TIL
#
miklb did mean to tag you GWG, but now that I did, that PR doesn't solve my issue unfortunately.
trip_ joined the channel
#
aaronpk I think he still was planning on working on a new fix for it given our discussion earlier
KartikPrabhu, [tantek] and [dougbeal] joined the channel
#
[dougbeal] There is a GitHub command line took called hub that I think handles pull requests.
#
aaronpk I remember that one, which is why I was surprised to see that `git` can do it itself
#
sknebel github just adds PRs to the main repository, nothing magic
#
aaronpk oh is it adding a branch named appropriately?
#
sknebel I think underlying a repo and all it's forks are actually stored in the same repo structure
#
sknebel and you get different "views" on it depending from which url you query
[cleverdevil] joined the channel
#
sknebel so yes, there's just a reference with the name pull/.../head that points to the right commit
#
sknebel so for the client its normal git protocol
#
aaronpk clever
davidmead and snarfed joined the channel