Skip to content
This repository has been archived by the owner on Nov 1, 2021. It is now read-only.

Server Push #1

Closed
caniszczyk opened this issue Dec 19, 2013 · 17 comments · May be fixed by #59
Closed

Server Push #1

caniszczyk opened this issue Dec 19, 2013 · 17 comments · May be fixed by #59

Comments

@caniszczyk
Copy link
Contributor

No description provided.

@eliziario
Copy link

It would be interesting also to implement server hint. A client could decide then whether to download the resource or not, and some optimizations could even be done by combining several resources discovered through hints by the client could later be retrieved in a single combined call for a package of resources.

@mazhao
Copy link

mazhao commented Dec 20, 2013

We are looking forward it

@blakewatters
Copy link
Contributor

Has any work been done on this yet? Alternately, are there any specs for the implementation?

We're going to be implementing this over the next couple of weeks and I'd love to make sure we deliver a merge-able patch.

@caniszczyk
Copy link
Contributor Author

As far as I know no, we definitely welcome contributions!

@goaway can comment some more potentially.

@blakewatters
Copy link
Contributor

We have a functional implementation of this over at the @layerhq fork. Pull request coming later this week

@blakewatters
Copy link
Contributor

We are up and running with an implementation of this. The PR against our fork is available at: https://github.com/layerhq/CocoaSPDY/pull/1

@goaway
Copy link
Collaborator

goaway commented Jun 20, 2014

@blakewatters this is awesome - I'll take a look this coming week. We have a couple unrelated updates to make, so it would be great to get this merged in to avoid conflicts.

@blakewatters
Copy link
Contributor

We are working with a pretty divergent branch on the Layer fork at this point. It would be great to circle up and figure out how much our touches we can get up stream.

Right now the full list of updates includes:

  • Server push
  • GCD concurrency changes
  • Addition of CLI automation for running the test suites (project changes to support this)

@milani
Copy link

milani commented Aug 25, 2014

Any update on this?

@blakewatters
Copy link
Contributor

The SPDY push is working like a champ for us. Would love to figure out how to get our bits merged

@kgoodier
Copy link
Contributor

We at @twitter are committed to implementing server push very soon, so updates & progress on this issue will be forthcoming. Stay tuned!

@blakewatters
Copy link
Contributor

@kgoodier you do realize that we’ve already implemented it, right?

@kgoodier
Copy link
Contributor

Yes, sorry -- by "implement" I just meant get it working, by whatever means necessary, first in the develop branch of CocoaSPDY, then in the master branch. Hopefully we can just merge, but since the branches have diverged it needs a close inspection.

@blakewatters
Copy link
Contributor

@kgoodier @goaway Clean patch against master for SPDY push available on #59

@plivesey
Copy link

plivesey commented Oct 1, 2015

Is there any update on this? It looks like @blakewatters work was abandoned?

@kgoodier
Copy link
Contributor

kgoodier commented Oct 6, 2015

I just issued a pull request (#136) for server push support based on work I did at the beginning of the year, which was itself based on work done by @blakewatters et al (#59).

This is fundamentally different from the approach taken in #59 by not exposing as much to the app. This means the app does not have to be aware of SPDY sessions and does not have to implement and register a custom delegate. Instead, the app uses the standard NSURL system for hooking up to pushed streams, with an NSNotification posted whenever new pushed requests are received.

You could still use a long-lived NSURLRequest and receive multiple push requests associated with it. This new implementation should provide all the same functionality that @blakewatters needed in his implementation, but will obviously require some code changes.

@kgoodier
Copy link
Contributor

#136 merged. Please open new issues to discuss future additions, changes, questions on use, etc.

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

Successfully merging a pull request may close this issue.

8 participants