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

Update cowboy bridge for compatibility with newer versions of cowboy #22

Closed
wants to merge 2 commits into from

Conversation

mgregson
Copy link

No description provided.

@choptastic
Copy link
Member

Hello Michael,

Thank you for the pull request. Cowboy 0.8 support will be merged in shortly, however @RomanShestakov beat you to the punch. The relevant commit can be found here: choptastic@78b8791

This change also includes significantly improved static file handling to change it from the hack I originally made for the cowboy 0.6 support, which will something that will now use sendfile rather than loading entire static files into memory.

choptastic@4ab0342

I appreciate the pull request, though :)

Edit: Typo fix s/not/now/

@choptastic choptastic closed this Apr 20, 2013
@mgregson
Copy link
Author

Awesome. Glad to hear it, and thanks for letting me know.

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.

None yet

2 participants