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

disable static file handler #2926

Merged
merged 1 commit into from Jul 27, 2017

Conversation

Projects
None yet
5 participants
@drujensen
Contributor

drujensen commented Jul 26, 2017

This disables the static file handler for Kemal. This provides a major performance enhancement and would be a standard configuration for RESTful servers.

@mention-bot

This comment has been minimized.

Show comment
Hide comment
@mention-bot

mention-bot Jul 26, 2017

Thanks @drujensen for contributing to The Framework Benchmarks! @Jesterovskiy, @jhass, @RX14 and @sdogruyol, code you've worked on has been modified. If you have the chance, please review. If you wish to unsubscribe from these notices, please open a Pull Request with the commit message [ci skip] and your github name added to the userBlacklist array in the .mention-bot file.

mention-bot commented Jul 26, 2017

Thanks @drujensen for contributing to The Framework Benchmarks! @Jesterovskiy, @jhass, @RX14 and @sdogruyol, code you've worked on has been modified. If you have the chance, please review. If you wish to unsubscribe from these notices, please open a Pull Request with the commit message [ci skip] and your github name added to the userBlacklist array in the .mention-bot file.

@drujensen

This comment has been minimized.

Show comment
Hide comment
@drujensen

drujensen Jul 26, 2017

Contributor

@RX14 @sdogruyol please review

@nbrady-techempower not sure if its possible, but this would be great to have in preview 2.

Contributor

drujensen commented Jul 26, 2017

@RX14 @sdogruyol please review

@nbrady-techempower not sure if its possible, but this would be great to have in preview 2.

@RX14

RX14 approved these changes Jul 26, 2017

@sdogruyol

This comment has been minimized.

Show comment
Hide comment
@sdogruyol

sdogruyol Jul 26, 2017

Contributor

This is great improvement, thanks @drujensen 👍

Contributor

sdogruyol commented Jul 26, 2017

This is great improvement, thanks @drujensen 👍

@nbrady-techempower nbrady-techempower merged commit 9f8c708 into TechEmpower:master Jul 27, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@drujensen drujensen deleted the drujensen:fix-kemal-performance branch Sep 10, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment