Skip to content

Conversation

berfarah
Copy link
Contributor

Hey @jodosha, added an example that I would use for a streaming response. I think a SSE example would be good too, but I don't have much experience with those.

Hey @jodosha, added an example that I would use for a streaming response. I think a SSE example would be good too, but I don't have much experience with those.
@jodosha
Copy link
Member

jodosha commented Oct 27, 2015

@berfarah Thank you, this looks good. Would you mind to describe which web servers work with it. From the description we communicate that WEBrick doesn't work, so we should provide a reliable alternative (aka standard) for this. What do you use?

@jodosha jodosha self-assigned this Oct 27, 2015
@jodosha jodosha added this to the 0.5.0 milestone Oct 27, 2015
Could also add passenger, unicorn and thin, but leaving it at puma for simplicity
@berfarah
Copy link
Contributor Author

@jodosha I've added puma to the readme, though I believe unicorn, thin and passenger would work as well. Would you like to provide multiple options or just the one?

@runlevel5
Copy link

@berfarah I think we should stick to puma, as it is recommended on Heroku and elsewhere too

@berfarah
Copy link
Contributor Author

@runlevel5
Copy link

good to me

runlevel5 pushed a commit that referenced this pull request Oct 28, 2015
Added Streaming Response example
@runlevel5 runlevel5 merged commit 0b7445f into hanami:master Oct 28, 2015
@berfarah berfarah deleted the patch-1 branch October 28, 2015 15:10
@jodosha
Copy link
Member

jodosha commented Oct 29, 2015

@berfarah Thank you! 👍

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

Successfully merging this pull request may close these issues.

3 participants