puma 0.9.0 → 0.9.1

data/Manifest.txt

@@ -8,6 +8,7 @@ Rakefile
 TODO
 bin/puma
 bin/pumactl
+examples/config.rb
 ext/puma_http11/PumaHttp11Service.java
 ext/puma_http11/ext_help.h
 ext/puma_http11/extconf.rb
@@ -22,9 +23,11 @@ ext/puma_http11/puma_http11.c
 lib/puma.rb
 lib/puma/app/status.rb
 lib/puma/cli.rb
+lib/puma/config.rb
 lib/puma/const.rb
 lib/puma/control_cli.rb
 lib/puma/events.rb
+lib/puma/jruby_restart.rb
 lib/puma/null_io.rb
 lib/puma/rack_patch.rb
 lib/puma/server.rb

data/examples/config.rb

@@ -0,0 +1,5 @@
+state_path "puma.state"
+activate_control_app
+
+rackup "test/lobster.ru"
+threads 3, 10

data/lib/puma/cli.rb

@@ -3,6 +3,7 @@ require 'uri'
 
 require 'puma/server'
 require 'puma/const'
+require 'puma/config'
 
 require 'rack/commonlogger'
 
@@ -10,9 +11,7 @@ module Puma
   # Handles invoke a Puma::Server in a command line style.
   #
   class CLI
-    DefaultTCPHost = "0.0.0.0"
-    DefaultTCPPort = 9292
-
+    DefaultRackup = "config.ru"
     IS_JRUBY = defined?(JRUBY_VERSION)
 
     # Create a new CLI object using +argv+ as the command line
@@ -81,8 +80,13 @@ module Puma
     end
 
     def restart!
-      Dir.chdir @restart_dir
-      Kernel.exec(*@restart_argv)
+      if IS_JRUBY
+        require 'puma/jruby_restart'
+        JRubyRestart.chdir_exec(@restart_dir, Gem.ruby, *@restart_argv)
+      else
+        Dir.chdir @restart_dir
+        Kernel.exec(*@restart_argv)
+      end
     end
 
     # Write +str+ to +@stdout+
@@ -104,14 +108,17 @@ module Puma
       @options = {
         :min_threads => 0,
         :max_threads => 16,
-        :quiet => false
+        :quiet => false,
+        :binds => []
       }
 
-      @binds = []
-
       @parser = OptionParser.new do |o|
         o.on "-b", "--bind URI", "URI to bind to (tcp:// and unix:// only)" do |arg|
-          @binds << arg
+          @options[:binds] << arg
+        end
+
+        o.on "-C", "--config PATH", "Load PATH as a config file" do |arg|
+          @options[:config_file] = arg
         end
 
         o.on "--pidfile PATH", "Use PATH as a pidfile" do |arg|
@@ -122,6 +129,19 @@ module Puma
           @options[:quiet] = true
         end
 
+        o.on "-S", "--state PATH", "Where to store the state details" do |arg|
+          @options[:state] = arg
+        end
+
+        o.on "--control URL", "The bind url to use for the control server",
+                              "Use 'auto' to use temp unix server" do |arg|
+          if arg
+            @options[:control_url] = arg
+          elsif IS_JRUBY
+            raise NotImplementedError, "No default url available on JRuby"
+          end
+        end
+
         o.on '-t', '--threads INT', "min:max threads to use (default 0:16)" do |arg|
           min, max = arg.split(":")
           if max
@@ -133,27 +153,6 @@ module Puma
           end
         end
 
-        o.on "-S", "--state PATH", "Where to store the state details" do |arg|
-          @options[:state] = arg
-        end
-
-        o.on "--status [URL]", "The bind url to use for the status server" do |arg|
-          if arg and arg != "@"
-            @options[:status_address] = arg
-          elsif IS_JRUBY
-            raise NotImplementedError, "No default url available on JRuby"
-          else
-            require 'tmpdir'
-
-            t = (Time.now.to_f * 1000).to_i
-            path = "#{Dir.tmpdir}/puma-status-#{t}-#{$$}"
-
-            @temp_status_path = path
-
-            @options[:status_address] = "unix://#{path}"
-          end
-        end
-
       end
 
       @parser.banner = "puma <options> <rackup file>"
@@ -168,7 +167,7 @@ module Puma
     # the rackup file, and set @app.
     #
     def load_rackup
-      @app, options = Rack::Builder.parse_file @rackup
+      @app, options = Rack::Builder.parse_file @options[:rackup]
       @options.merge! options
 
       options.each do |key,val|
@@ -195,9 +194,7 @@ module Puma
       if path = @options[:state]
         state = { "pid" => Process.pid }
 
-        if url = @options[:status_address]
-          state["status_address"] = url
-        end
+        state["config"] = @config
 
         File.open(path, "w") do |f|
           f.write state.to_yaml
@@ -208,6 +205,18 @@ module Puma
     # :nodoc:
     def parse_options
       @parser.parse! @argv
+
+      @config = Puma::Configuration.new @options
+      @config.load
+
+      unless @options[:rackup]
+        @options[:rackup] = @argv.shift || DefaultRackup
+      end
+
+      if @options[:control_url] == "auto"
+        path = @temp_status_path = Configuration.temp_path
+        @options[:control_url] = "unix://#{path}"
+      end
     end
 
     # Parse the options, load the rackup, start the server and wait
@@ -216,10 +225,10 @@ module Puma
     def run
       parse_options
 
-      @rackup = @argv.shift || "config.ru"
+      rackup = @options[:rackup]
 
-      unless File.exists?(@rackup)
-        raise "Missing rackup file '#{@rackup}'"
+      unless File.exists?(rackup)
+        raise "Missing rackup file '#{rackup}'"
       end
 
       load_rackup
@@ -230,11 +239,6 @@ module Puma
         @app = Rack::CommonLogger.new(@app, STDOUT)
       end
 
-      if @binds.empty?
-        @options[:Host] ||= DefaultTCPHost
-        @options[:Port] ||= DefaultTCPPort
-      end
-
       min_t = @options[:min_threads]
       max_t = @options[:max_threads]
 
@@ -245,12 +249,7 @@ module Puma
       log "Puma #{Puma::Const::PUMA_VERSION} starting..."
       log "* Min threads: #{min_t}, max threads: #{max_t}"
 
-      if @options[:Host]
-        log "* Listening on tcp://#{@options[:Host]}:#{@options[:Port]}"
-        server.add_tcp_listener @options[:Host], @options[:Port]
-      end
-
-      @binds.each do |str|
+      @options[:binds].each do |str|
         uri = URI.parse str
         case uri.scheme
         when "tcp"
@@ -268,7 +267,7 @@ module Puma
 
       @server = server
 
-      if str = @options[:status_address]
+      if str = @options[:control_url]
         require 'puma/app/status'
 
         uri = URI.parse str

data/lib/puma/config.rb

@@ -0,0 +1,98 @@
+module Puma
+  class Configuration
+    DefaultTCPHost = "0.0.0.0"
+    DefaultTCPPort = 9292
+
+    def initialize(options)
+      @options = options
+      @options[:binds] ||= []
+    end
+
+    attr_reader :options
+
+    def load
+      if path = @options[:config_file]
+        instance_eval File.read(path), path, 1
+      end
+
+      # Rakeup default option support
+      if host = @options[:Host]
+        port = @options[:Port] || DefaultTCPPort
+
+        @options[:binds] << "tcp://#{host}:#{port}"
+      end
+
+      if @options[:binds].empty?
+        @options[:binds] << "tcp://#{DefaultTCPHost}:#{DefaultTCPPort}"
+      end
+    end
+
+    def self.temp_path
+      require 'tmpdir'
+
+      t = (Time.now.to_f * 1000).to_i
+      "#{Dir.tmpdir}/puma-status-#{t}-#{$$}"
+    end
+
+    # Use +obj+ or +block+ as the Rack app. This allows a config file to
+    # be the app itself.
+    #
+    def app(obj=nil, &block)
+      obj ||= block
+
+      raise "Provide either a #call'able or a block" unless obj
+
+      @options[:app] = obj
+    end
+
+    # Start the Puma control rack app on +url+. This app can be communicated
+    # with to control the main server.
+    #
+    def activate_control_app(url="auto")
+      @options[:control_url] = url
+    end
+
+    # Bind the server to +url+. tcp:// and unix:// are the only accepted
+    # protocols.
+    #
+    def bind(url)
+      @options[:binds] << url
+    end
+
+    # Store the pid of the server in the file at +path+.
+    def pidfile(path)
+      @options[:pidfile] = path
+    end
+
+    # Disable request logging.
+    #
+    def quiet
+      @options[:quiet] = true
+    end
+
+    # Load +path+ as a rackup file.
+    #
+    def rackup(path)
+      @options[:rackup] = path.to_s
+    end
+
+    # Configure +min+ to be the minimum number of threads to use to answer
+    # requests and +max+ the maximum.
+    #
+    def threads(min, max)
+      if min > max
+        raise "The minimum number of threads must be less than the max"
+      end
+
+      @options[:min_threads] = min
+      @options[:max_threads] = max
+    end
+
+    # Use +path+ as the file to store the server info state. This is
+    # used by pumactl to query and control the server.
+    #
+    def state_path(path)
+      @options[:state] = path.to_s
+    end
+  end
+end

data/lib/puma/const.rb

@@ -71,7 +71,7 @@ module Puma
 
     PATH_INFO = 'PATH_INFO'.freeze
 
-    PUMA_VERSION = VERSION = "0.9.0".freeze
+    PUMA_VERSION = VERSION = "0.9.1".freeze
 
     PUMA_TMP_BASE = "puma".freeze
 

data/lib/puma/control_cli.rb

@@ -1,6 +1,8 @@
 require 'optparse'
 
 require 'puma/const'
+require 'puma/config'
+
 require 'yaml'
 require 'uri'
 
@@ -22,7 +24,7 @@ module Puma
     end
 
     def connect
-      if str = @state['status_address']
+      if str = @config.options[:control_url]
         uri = URI.parse str
         case uri.scheme
         when "tcp"
@@ -44,6 +46,7 @@ module Puma
       @parser.parse! @argv
 
       @state = YAML.load_file(@path)
+      @config = @state['config']
 
       cmd = @argv.shift
 
@@ -101,8 +104,8 @@ module Puma
 
     def command_stats
       sock = connect
-      s << "GET /stats HTTP/1.0\r\n\r\n"
-      rep = s.read
+      sock << "GET /stats HTTP/1.0\r\n\r\n"
+      rep = sock.read
 
       body = rep.split("\r\n").last
 

data/lib/puma/jruby_restart.rb

@@ -0,0 +1,22 @@
+require 'ffi'
+
+module Puma
+  module JRubyRestart
+    extend FFI::Library
+    ffi_lib 'c'
+
+    attach_function :execlp, [:string, :varargs], :int
+    attach_function :chdir, [:string], :int
+
+    def self.chdir_exec(dir, cmd, *argv)
+      chdir(dir)
+      argv.unshift(cmd)
+      argv = ([:string] * argv.size).zip(argv).flatten
+      argv <<:int
+      argv << 0
+      execlp(cmd, *argv)
+      raise SystemCallError.new(FFI.errno)
+    end
+  end
+end
+

data/lib/puma/thread_pool.rb

@@ -137,7 +137,9 @@ module Puma
         @todo << Stop
       end
 
-      @workers.each { |w| w.join }
+      # Use this instead of #each so that we don't stop in the middle
+      # of each and see a mutated object mid #each
+      @workers.first.join until @workers.empty?
 
       @spawned = 0
       @workers = []

data/puma.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |s|
   s.name = "puma"
-  s.version = "0.9.0"
+  s.version = "0.9.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Evan Phoenix"]
@@ -12,11 +12,11 @@ Gem::Specification.new do |s|
   s.executables = ["puma", "pumactl"]
   s.extensions = ["ext/puma_http11/extconf.rb"]
   s.extra_rdoc_files = ["History.txt", "Manifest.txt"]
-  s.files = ["COPYING", "Gemfile", "History.txt", "LICENSE", "Manifest.txt", "README.md", "Rakefile", "TODO", "bin/puma", "bin/pumactl", "ext/puma_http11/PumaHttp11Service.java", "ext/puma_http11/ext_help.h", "ext/puma_http11/extconf.rb", "ext/puma_http11/http11_parser.c", "ext/puma_http11/http11_parser.h", "ext/puma_http11/http11_parser.java.rl", "ext/puma_http11/http11_parser.rl", "ext/puma_http11/http11_parser_common.rl", "ext/puma_http11/org/jruby/puma/Http11.java", "ext/puma_http11/org/jruby/puma/Http11Parser.java", "ext/puma_http11/puma_http11.c", "lib/puma.rb", "lib/puma/app/status.rb", "lib/puma/cli.rb", "lib/puma/const.rb", "lib/puma/control_cli.rb", "lib/puma/events.rb", "lib/puma/null_io.rb", "lib/puma/rack_patch.rb", "lib/puma/server.rb", "lib/puma/thread_pool.rb", "lib/rack/handler/puma.rb", "puma.gemspec", "test/ab_rs.rb", "test/lobster.ru", "test/mime.yaml", "test/test_app_status.rb", "test/test_cli.rb", "test/test_http10.rb", "test/test_http11.rb", "test/test_persistent.rb", "test/test_rack_handler.rb", "test/test_rack_server.rb", "test/test_thread_pool.rb", "test/test_unix_socket.rb", "test/test_ws.rb", "test/testhelp.rb", "tools/trickletest.rb"]
+  s.files = ["COPYING", "Gemfile", "History.txt", "LICENSE", "Manifest.txt", "README.md", "Rakefile", "TODO", "bin/puma", "bin/pumactl", "examples/config.rb", "ext/puma_http11/PumaHttp11Service.java", "ext/puma_http11/ext_help.h", "ext/puma_http11/extconf.rb", "ext/puma_http11/http11_parser.c", "ext/puma_http11/http11_parser.h", "ext/puma_http11/http11_parser.java.rl", "ext/puma_http11/http11_parser.rl", "ext/puma_http11/http11_parser_common.rl", "ext/puma_http11/org/jruby/puma/Http11.java", "ext/puma_http11/org/jruby/puma/Http11Parser.java", "ext/puma_http11/puma_http11.c", "lib/puma.rb", "lib/puma/app/status.rb", "lib/puma/cli.rb", "lib/puma/config.rb", "lib/puma/const.rb", "lib/puma/control_cli.rb", "lib/puma/events.rb", "lib/puma/jruby_restart.rb", "lib/puma/null_io.rb", "lib/puma/rack_patch.rb", "lib/puma/server.rb", "lib/puma/thread_pool.rb", "lib/rack/handler/puma.rb", "puma.gemspec", "test/ab_rs.rb", "test/lobster.ru", "test/mime.yaml", "test/test_app_status.rb", "test/test_cli.rb", "test/test_http10.rb", "test/test_http11.rb", "test/test_persistent.rb", "test/test_rack_handler.rb", "test/test_rack_server.rb", "test/test_thread_pool.rb", "test/test_unix_socket.rb", "test/test_ws.rb", "test/testhelp.rb", "tools/trickletest.rb"]
   s.rdoc_options = ["--main", "README.md"]
   s.require_paths = ["lib"]
   s.rubyforge_project = "puma"
-  s.rubygems_version = "1.8.10"
+  s.rubygems_version = "1.8.12"
   s.summary = "Puma is a small library that provides a very fast and concurrent HTTP 1.1 server for Ruby web applications"
   s.test_files = ["test/test_app_status.rb", "test/test_cli.rb", "test/test_http10.rb", "test/test_http11.rb", "test/test_persistent.rb", "test/test_rack_handler.rb", "test/test_rack_server.rb", "test/test_thread_pool.rb", "test/test_unix_socket.rb", "test/test_ws.rb"]
 

data/test/test_cli.rb

@@ -28,13 +28,13 @@ class TestCLI < Test::Unit::TestCase
   end
 
   unless defined? JRUBY_VERSION
-  def test_status
+  def test_control
     url = "unix://#{@tmp_path}"
 
     sin = StringIO.new
     sout = StringIO.new
 
-    cli = Puma::CLI.new ["-b", "unix://#{@tmp_path2}", "--status", url, "test/lobster.ru"], sin, sout
+    cli = Puma::CLI.new ["-b", "unix://#{@tmp_path2}", "--control", url, "test/lobster.ru"], sin, sout
     cli.parse_options
 
     t = Thread.new { cli.run }
@@ -51,13 +51,13 @@ class TestCLI < Test::Unit::TestCase
     t.join
   end
 
-  def test_status_stop
+  def test_control_stop
     url = "unix://#{@tmp_path}"
 
     sin = StringIO.new
     sout = StringIO.new
 
-    cli = Puma::CLI.new ["-b", "unix://#{@tmp_path2}", "--status", url, "test/lobster.ru"], sin, sout
+    cli = Puma::CLI.new ["-b", "unix://#{@tmp_path2}", "--control", url, "test/lobster.ru"], sin, sout
     cli.parse_options
 
     t = Thread.new { cli.run }
@@ -73,9 +73,9 @@ class TestCLI < Test::Unit::TestCase
     t.join
   end
 
-  def test_tmp_status
+  def test_tmp_control
     url = "tcp://127.0.0.1:8232"
-    cli = Puma::CLI.new ["--state", @tmp_path, "--status"]
+    cli = Puma::CLI.new ["--state", @tmp_path, "--control", "auto"]
     cli.parse_options
     cli.write_state
 
@@ -83,7 +83,7 @@ class TestCLI < Test::Unit::TestCase
 
     assert_equal Process.pid, data["pid"]
 
-    url = data["status_address"]
+    url = data["config"].options[:control_url]
 
     m = %r!unix://(.*)!.match(url)
 
@@ -93,13 +93,13 @@ class TestCLI < Test::Unit::TestCase
 
   def test_state
     url = "tcp://127.0.0.1:8232"
-    cli = Puma::CLI.new ["--state", @tmp_path, "--status", url]
+    cli = Puma::CLI.new ["--state", @tmp_path, "--control", url]
     cli.parse_options
     cli.write_state
 
     data = YAML.load_file(@tmp_path)
 
     assert_equal Process.pid, data["pid"]
-    assert_equal url, data["status_address"]
+    assert_equal url, data["config"].options[:control_url]
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: puma
 version: !ruby/object:Gem::Version 
-  hash: 59
+  hash: 57
   prerelease: 
   segments: 
   - 0
   - 9
-  - 0
-  version: 0.9.0
+  - 1
+  version: 0.9.1
 platform: ruby
 authors: 
 - Evan Phoenix
@@ -103,6 +103,7 @@ files:
 - TODO
 - bin/puma
 - bin/pumactl
+- examples/config.rb
 - ext/puma_http11/PumaHttp11Service.java
 - ext/puma_http11/ext_help.h
 - ext/puma_http11/extconf.rb
@@ -117,9 +118,11 @@ files:
 - lib/puma.rb
 - lib/puma/app/status.rb
 - lib/puma/cli.rb
+- lib/puma/config.rb
 - lib/puma/const.rb
 - lib/puma/control_cli.rb
 - lib/puma/events.rb
+- lib/puma/jruby_restart.rb
 - lib/puma/null_io.rb
 - lib/puma/rack_patch.rb
 - lib/puma/server.rb