Commit 1840884e04cdd1a1f4e9eab8b181f7561466416d to macournoyer's thin

1840884
all branches
- libev
- master
- stable
all tags
- v0.8.2
- v0.8.1
- v0.7.1
- v0.7.0
- v0.6.3
- v0.6.2
- v0.6.1
- v0.6.0
- v0.5.3
- v0.5.2
comments

macournoyer / thin

Description:	A very fast & simple Ruby web server
Homepage:	http://code.macournoyer.com/thin/
Clone URL:	git://github.com/macournoyer/thin.git Give this clone URL to anyone. `git clone git://github.com/macournoyer/thin.git`
Search Repo:

Add error handling to parser

macournoyer (author)

Thu Mar 27 21:51:50 -0700 2008

commit 1840884e04cdd1a1f4e9eab8b181f7561466416d
tree 3c0b88114d457c59746c1d83c59aae416e0d77fd
parent 2cd6746715f3743148777db85922a20b55dc2189

Rakefile
ext/thin_backend/connection.c
ext/thin_backend/parser.c
ext/thin_backend/parser.h
ext/thin_backend/parser.rl
ext/thin_backend/parser_callbacks.c
ext/thin_backend/test.rb
ext/thin_backend/thin.h
tasks/ext.rake
test/status_test.c

Rakefile

0
@@ -13,6 +13,10 @@ task :default => :spec
0
 ext_task :thin_parser
0
 ext_task :thin_backend
0
 
0
+ragel_task 'ext/thin_backend', 'parser.rl', 'parser.c'
0
+
0
 task :test do
0
-  cd 'test' { sh 'make test' }
0
-end
0
\ No newline at end of file
0
+  cd 'test' do
0
+    sh 'make test'
0
+  end
0
+end

ext/thin_backend/connection.c

0
@@ -115,7 +115,16 @@ void thin_connection_readable_cb(EV_P_ struct ev_io *watcher, int revents)
0
                             connection->parser.nread);
0
   
0
   connection->parser.nread = len;
0
-
0
+  
0
+  /* parser error */
0
+  if (http_parser_has_error(&connection->parser)) {
0
+    rb_funcall(connection->backend->obj, rb_intern("log_error"), 1,
0
+               rb_exc_new(rb_eRuntimeError, "Invalid request", 15));
0
+    thin_connection_close(connection);
0
+    return;
0
+  }
0
+  
0
+  /* TODO store big body in tempfile */
0
   if (http_parser_is_finished(&connection->parser) &&
0
       connection->read_buffer.len >= connection->content_length) {
0

ext/thin_backend/parser.c