Commits
- Commit:
807869c14ee57aaa6035e6cba0df5d1369ced9ba
- From:
- Omar Polo <op@omarpolo.com>
- Date:
print the error too if we can't open a directory
It's not intuitive to print
open ... for domain xyz
it doesn't convey that the open failed.
now it appends the error string, at least the user can understand that
something went wrong.
reported by cage on irc, thanks!
- Commit:
492a274fd712e4589669254be327897868e44812
- From:
- Omar Polo <op@omarpolo.com>
- Date:
add compat for sys/tree.h
- Commit:
207b3e80d867693ff74cf99c84f7dd41386adba1
- From:
- Omar Polo <op@omarpolo.com>
- Date:
Store clients inside a splay tree
From day one we've been using a static array of client struct to hold
the clients data. This has variuos drawbacks, among which:
* reuse of the storage ("shades of heartbleed")
* maximum fixed amount of clients connected at the same time
* bugs are harder to debug
The last point in particular is important because if we mess the client
ids, or try to execute some functions (e.g. the various fcgi_*) after a
client has been disconnected, it's harder to "see" this "use after
free"-tier kind of bug.
Now I'm using a splay tree to hold the data about the live connections.
Each client' data is managed by malloc. If we try to access a client
data after the disconnection we'll probably crash with a SIGSEGV and
find the bug is more easy.
Performance-wise the connection phase should be faster since we don't
have to loop anymore to find an empty spot in the clients array, but
some operations could be slightly slower (compare the O(1) access in an
array with a SPLAY_FIND operation -- still be faster than O(n) thought.)
- Commit:
4cd25209651f224be8c34d6006ef689963ce37d5
- From:
- Omar Polo <op@omarpolo.com>
- Date:
one FastCGI connection per client
FastCGI is designed to multiplex requests over a single connection, so
ideally the server can open only one connection per worker to the
FastCGI application and that's that.
Doing this kind of multiplexing makes the code harder to follow and
easier to break/leak etc on the gmid side however. OpenBSD' httpd
seems to open one connection per client, so why can't we too?
One connection per request is still way better (lighter) than using
CGI, and we can avoid all the pitfalls of the multiplexing (keeping
track of "live ids", properly shut down etc...)
- Commit:
3096da4ef4418dc57f3e0b1fb1f89dceb2ca426a
- From:
- Omar Polo <op@omarpolo.com>
- Date:
allow to run only a subset of the runtime tests
with
make TESTS='test_1 test_2 ...' regress
now it's possible to run only that specified subset of tests. It's
really useful during debugging :)
- Commit:
e4daebe44aedd66413f82319252a7e579133945d
- From:
- Omar Polo <op@omarpolo.com>
- Date:
plug a memory leak
c->req is set in client_read but never deallocated
- Commit:
807a80cb9efdf631c3717fdca884bd0119493d45
- From:
- Omar Polo <op@omarpolo.com>
- Date:
fmt
- Commit:
b4c6cd976806864f433211d265065abafbd49316
- From:
- Omar Polo <op@omarpolo.com>
- Date:
add the upload target to ease publishing the site
- Commit:
9212cf1ba9487a66662df76b95591187dface70f
- From:
- Omar Polo <op@omarpolo.com>
- Date:
[gemini] tweak the contrib page
I find it more readable with some empty lines here and there
- Commit:
eb82dcfbf4a4013f0ad3ba359c32c79c87953e25
- From:
- Omar Polo <op@omarpolo.com>
- Date:
improve the service file usage instructions
Thanks Martin for providing these information :)
- Commit:
12866f1911ebefc40cf4f988cd59d620e8b50a96
- From:
- Omar Polo <op@omarpolo.com>
- Date:
add targets to serve the site locally
- Commit:
ae6870fa3bf25561a3f6bd8465ba86307af5d5bb
- From:
- Omar Polo <op@omarpolo.com>
- Date:
import the capsule/website
- Commit:
568419b2c1f71620095acd9bf3be6aaa2bbe43ee
- From:
- Omar Polo <op@omarpolo.com>
- Date:
add .cirrus.yml
Add a cirrus CI config file that runs the regression suite on linux
amd64/aarch64 and on freebsd.
- Commit:
6e0f14d51ef1971893bb094c873ddde86d0cae61
- From:
- Omar Polo <op@omarpolo.com>
- Date:
re-add sha script; it's used in the Makefile
While there, use it in the tests too
- Commit:
2072343d6ba88a37ede0c4ae2791328880cdfacf
- From:
- Omar Polo <op@omarpolo.com>
- Date:
sync changelog