Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Render syndication links on feed pages, or support ETags/If-Modified-Since #1309

Closed
snarfed opened this issue Feb 26, 2016 · 13 comments
Closed

Comments

@snarfed
Copy link

snarfed commented Feb 26, 2016

we've been working on optimizing bridgy to cut costs recently (snarfed/bridgy#578 , snarfed/bridgy#624), and one of its big sources of load currently is refetching home pages and permalinks to check for new/updated syndication links.

Known and WordPress are the two biggest CMSes bridgy sees by far. if you all were able to render synd links inside entries in feeds, that would let us skip fetching the permalinks, which would be a huge help.

alternatively, if you supported ETags/If-Modified-Since, we could use them to make them permalink fetches and processing much lighter. a bit more work for us, and not quite as big a win, but still something.

cc @kylewm. thanks in advance! hope you all are good. oh, and feel free to split this into two issues if you want, obviously.

@benwerd
Copy link
Member

benwerd commented Feb 26, 2016

Seems totally reasonable. Are you scanning the mf2 or the RSS feeds?

On Friday, February 26, 2016, Ryan Barrett notifications@github.com wrote:

we've been working on optimizing bridgy to cut costs recently (
snarfed/bridgy#578 snarfed/bridgy#578 ,
snarfed/bridgy#624 snarfed/bridgy#624), and
one of its big sources of load currently is refetching home pages and
permalinks to check for new/updated syndication links.

Known and WordPress are the two biggest CMSes bridgy sees by far. if you
all were able to render synd links inside entries in feeds, that would let
us skip fetching the permalinks, which would be a huge help.

alternatively, if you supported ETags/If-Modified-Since, we could use them
to make them permalink fetches and processing much lighter. a bit more work
for us, and not quite as big a win, but still something.

cc @kylewm https://github.com/kylewm. thanks in advance! hope you all
are good. oh, and feel free to split this into two issues if you want,
obviously.


Reply to this email directly or view it on GitHub
#1309.

Ben Werdmuller
http://goog_1933028737
benwerd.com | werd.io

+1 (312) 488-9373

@snarfed
Copy link
Author

snarfed commented Feb 26, 2016

thanks! mf2, from the rendered pages, usually starting from the home page.

@benwerd
Copy link
Member

benwerd commented Feb 26, 2016

OK. So would mf2 syndication links in a sub div for each entry hidden with
display:none be acceptable?

On Friday, February 26, 2016, Ryan Barrett notifications@github.com wrote:

thanks! mf2, from the rendered pages, usually starting from the home page.


Reply to this email directly or view it on GitHub
#1309 (comment).

Ben Werdmuller
http://goog_1933028737
benwerd.com | werd.io

+1 (312) 488-9373

@snarfed
Copy link
Author

snarfed commented Feb 26, 2016

that'd be perfect!

benwerd added a commit that referenced this issue Feb 26, 2016
Refs #1309

cc @snarfed @kylewm - does this meet the need? This is live on
http://werd.io now.
@benwerd
Copy link
Member

benwerd commented Feb 26, 2016

Actually disregard last - it's probably outside the div you need it to be inside.

@benwerd
Copy link
Member

benwerd commented Feb 26, 2016

... Or on reflection, is it? Let me know.

@snarfed
Copy link
Author

snarfed commented Feb 26, 2016

looks like it works perfectly: http://mf2.kylewm.com/?url=http://werd.io/

thanks again!

@benwerd
Copy link
Member

benwerd commented Feb 26, 2016

Hooray! Okay, closing this out. We need to update the hosted service soon, so this will make its way to most Known sites before too long. There will also be a release at the beginning of March.

@benwerd benwerd closed this as completed Feb 26, 2016
@kylewm
Copy link
Collaborator

kylewm commented Feb 26, 2016

This is a win for woodwind too fwiw :)

@kylewm
Copy link
Collaborator

kylewm commented Feb 26, 2016

Hey that was almost a palindrome

@kylewm kylewm changed the title Render subdomain links on feed pages, or support ETags/If-Modified-Since Render syndication links on feed pages, or support ETags/If-Modified-Since Feb 26, 2016
@kylewm
Copy link
Collaborator

kylewm commented Feb 26, 2016

or maybe not quite a win... since rel=syndication is page-scoped, it's bad to have it in the feed. or we could talk about making mf2py scope rel=syndication to the block like it does for rel=bookmark

screenshot from 2016-02-26 11-52-18

@kylewm kylewm reopened this Feb 26, 2016
@benwerd
Copy link
Member

benwerd commented Feb 26, 2016

Eek. What do you think the better markup would be?

On Friday, February 26, 2016, Kyle Mahan notifications@github.com wrote:

or maybe not quite a win... since rel=syndication is page-scoped, it's
bad to have it in the feed.

[image: screenshot from 2016-02-26 11-52-18]
https://cloud.githubusercontent.com/assets/950127/13363690/96ba0b38-dc7f-11e5-824e-e4e98f466ffa.png


Reply to this email directly or view it on GitHub
#1309 (comment).

Ben Werdmuller
http://goog_1933028737
benwerd.com | werd.io

+1 (312) 488-9373

@kylewm
Copy link
Collaborator

kylewm commented Feb 26, 2016

Just like it is now but without rel="syndication"... (btw, this is always a hassle, I had it with Red Wind, and with adactio.com. Maybe a reason to change the spec)

<a href="https://twitter.com/benwerd/status/703279057329238016" class="u-syndication twitter"><i class="fa fa-twitter"></i>
 @benwerd</a>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants