diamant 0.0.2 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 72a20983fb5689076b1da91f753515fb251f96c73917628ba5d2e975d34c526c
-  data.tar.gz: efed078e4f3f0f2a556da98b2308fdc2d6d2af927b4e7ef61901b08490e4ef55
+  metadata.gz: c406e42cc1a01c8fd089f314e6e020bc7fcb94658f6d67fc20d3d77c3ba7f842
+  data.tar.gz: '08c40e5c6155396c1c9681146438bdfb68465853e3c51a71d2e112e0485f2840'
 SHA512:
-  metadata.gz: c15a92708e8b9bf8ebc746dd850e4f500affbac1fe32e29c0ee2b73efd998b12865ba2e6f68dd130bf42f3f042eed267852ed9a545bcc15797264ccbd39edc3f
-  data.tar.gz: '084cca645aa707d1140a787a35a8a548a2385bb9dfc4aa25110df0f8b75875238595accf4feb21bcc4f7cc9ec5d5e431ec75ace67943f0d48d3abae14f8674c7'
+  metadata.gz: 184a94626fa34d1e117cac0ab8d8814b520a12eb904b8804aad5466f49eea638ed4e02082148da504dada15b619ef67684465acf709a1b07bb8de3ab936d9448
+  data.tar.gz: 2641c99a8f263c33bd93c67363be5209d424d2ace599d52abc58ca16056f9f9c786a411dab89b77d658e531b0f405385838e768f0a0b53b145215aeafd3e57e4

data/lib/diamant.rb

@@ -11,6 +11,7 @@ require 'uri/gemini'
 
 require 'diamant/version'
 require 'diamant/mimetype'
+require 'diamant/response'
 
 module Diamant
   # Runs the server request/answer loop.
@@ -25,34 +26,32 @@ module Diamant
     def start
       tcp_serv = TCPServer.new @bind, @port
       ssl_serv = OpenSSL::SSL::SSLServer.new tcp_serv, ssl_context
+      main_loop(ssl_serv)
+    ensure
+      ssl_serv&.shutdown
+    end
+
+    private
+
+    include Diamant::Response
+
+    def main_loop(ssl_serv)
       loop do
         Thread.new(ssl_serv.accept) do |client|
           handle_client(client)
           client.close
         end
+      rescue OpenSSL::SSL::SSLError => e
+        # Do not even try to answer anything as the socket cannot be
+        # built. This will abruptly interrupt the connection from a client
+        # point of view, which must deal with it. Only keep a trace for us.
+        @logger.error(
+          format('SSLError: %<cause>s',
+                 cause: e.message.sub(/.*state=error: (.+)\Z/, '\1'))
+        )
       rescue Interrupt
         break
       end
-    ensure
-      ssl_serv&.shutdown
-    end
-
-    private
-
-    def reject_request?(sock, current_load)
-      # Accept only 10 thread with no restriction
-      return false if current_load < 11
-      # Seppuku
-      raise 'Server is under heavy load' if current_load > 1965
-      if current_load > 42
-        @logger.warn '41 - Too much threads...'
-        sock.puts "41 See you soon...\r\n"
-        return true
-      end
-      # Please wait a little
-      @logger.warn '44 5 - Too much threads...'
-      sock.puts "44 5\r\n"
-      true
     end
 
     def handle_client(client)
@@ -67,34 +66,6 @@ module Diamant
       end
     end
 
-    def read_file(client)
-      r = Net::GeminiRequest.read_new(client)
-      [r.uri, route(r.path)]
-    rescue Net::GeminiBadRequest
-      [nil, ["59\r\n"]]
-    end
-
-    def build_response(route)
-      info = Diamant::MimeType.new(route)
-      answer = IO.readlines route, chomp: true
-      answer.prepend "20 #{info.content_type}"
-    rescue Diamant::MimeError
-      ['50 Not a supported file!']
-    end
-
-    def route(path)
-      # In any case, remove the / prefix
-      route = File.expand_path path.delete_prefix('/'), Dir.pwd
-      # We better should use some sort of chroot...
-      unless route.start_with?(Dir.pwd)
-        @logger.warn "Bad attempt to get something out of public_dir: #{route}"
-        return ['51 Not found!']
-      end
-      route << '/index.gmi' if File.directory?(route)
-      return ['51 Not found!'] unless File.exist?(route)
-      build_response route
-    end
-
     def ssl_context
       ssl_context = OpenSSL::SSL::SSLContext.new
       ssl_context.min_version = OpenSSL::SSL::TLS1_2_VERSION

data/lib/diamant/response.rb

@@ -0,0 +1,52 @@
+# frozen_string_literal: true
+
+module Diamant
+  # Methods to generate requests responses
+  module Response
+    private
+
+    def reject_request?(sock, current_load)
+      # Accept only 10 thread with no restriction
+      return false if current_load < 11
+      # Seppuku
+      raise 'Server is under heavy load' if current_load > 1965
+      if current_load > 42
+        @logger.warn '41 - Too much threads...'
+        sock.puts "41 See you soon...\r\n"
+        return true
+      end
+      # Please wait a little
+      @logger.warn '44 5 - Too much threads...'
+      sock.puts "44 5\r\n"
+      true
+    end
+
+    def read_file(client)
+      r = Net::GeminiRequest.read_new(client)
+      [r.uri, route(r.path)]
+    rescue Net::GeminiBadRequest
+      [nil, ["59\r\n"]]
+    end
+
+    def route(path)
+      # In any case, remove the / prefix
+      route = File.expand_path path.delete_prefix('/'), Dir.pwd
+      # We better should use some sort of chroot...
+      unless route.start_with?(Dir.pwd)
+        @logger.warn "Bad attempt to get something out of public_dir: #{route}"
+        return ['51 Not found!']
+      end
+      route << '/index.gmi' if File.directory?(route)
+      return ['51 Not found!'] unless File.exist?(route)
+      build_response route
+    end
+
+    def build_response(route)
+      info = Diamant::MimeType.new(route)
+      answer = IO.readlines route, chomp: true
+      answer.prepend "20 #{info.content_type}"
+    rescue Diamant::MimeError
+      ['50 Not a supported file!']
+    end
+  end
+end

data/lib/diamant/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Diamant
-  VERSION = '0.0.2'
+  VERSION = '0.0.3'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: diamant
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors:
 - Étienne Deparis
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-11-20 00:00:00.000000000 Z
+date: 2020-12-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ruby-net-text
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.0.2
+        version: 0.0.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.0.2
+        version: 0.0.3
 - !ruby/object:Gem::Dependency
   name: byebug
   requirement: !ruby/object:Gem::Requirement
@@ -165,6 +165,7 @@ files:
 - lib/diamant.rb
 - lib/diamant/cert_generator.rb
 - lib/diamant/mimetype.rb
+- lib/diamant/response.rb
 - lib/diamant/version.rb
 homepage: https://git.umaneti.net/diamant/about/
 licenses: