Welcome! Log In Create A New Profile

Advanced

Request for Review: nginx module to strip whitespace from served pages.

Vedang
April 02, 2011 10:28AM
Hi All,

I've written a module to strip whitespace from text served by nginx in order
to reduce bandwidth consumed. I'm currently testing my module on a staging
server with nginx 0.7.67. However, since this is the first nginx-module that
I've written, I would be grateful if someone could review my code and point
out any mistakes I might have made from ignorance/inexperience.

The code for my module is at https://github.com/vedang/nginx-nonewlines. I
have based it on a function used by reddit, the code for which can be found
here:
https://github.com/reddit/reddit/blob/master/r2/r2/lib/c/filters.c#L216. Any
suggestions would be highly welcome. Thanks!


--
Unix is simple. It takes a genius to understand it's simplicity. -
Anon
People think it must be fun to be a super genius, but they don't realize how
hard it is to put up with all the idiots in the world. - Calvin.

Cheers,
Vedang.

Software Developer,
Infinitely Beta.
http://twitter.com/vedang
http://vedang.me
_______________________________________________
nginx mailing list
nginx@nginx.org
http://nginx.org/mailman/listinfo/nginx
Subject Author Posted

Request for Review: nginx module to strip whitespace from served pages.

Vedang April 02, 2011 10:28AM



Sorry, only registered users may post in this forum.

Click here to login

Online Users

Guests: 144
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready