websocket-server 1.0.1-java → 1.1.2-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7670d3a4e743e533e42467d4047741a0ea833cc4e7ea4d1855a1ed9b76138560
-  data.tar.gz: d1d6b8efb9502a45b44d56267d8a76fc036d46e4169c15cf93fcc605202ded8c
+  metadata.gz: 1a82a08a0eb5e690dcea50d8df9c9173fc9ee89dcb533e856b70c4a6622a5c11
+  data.tar.gz: 561c75644b76a40ed767a5c4e03d7d35dfb606c1819d39f08449b0c4283620ff
 SHA512:
-  metadata.gz: a7cf7669311ab73b6a1905cda6d8112052b32ddbd7da3ccfb6bcf36e36e3f1d66b8fabf11da1b0abe0dc29a61c96f10506c1a8c750f95a45a18f494e711ec24f
-  data.tar.gz: 16e0b9f2221cd9a83de57266561ea8797f40ac7d6d717e6b67fef1e078f045a5b2a5605c25c5670bc6fdde6c5db48610b85786f0a5c1a79d2a7df20c366ef6d6
+  metadata.gz: '083153e26d8f09e3cf8c9667bb02970d5861bba51d6390b9d02b8a5a5950c8c5e47afb172822b44e2e2e14810f90112314b87d3a5fbef201bbec1c8f2d4b46d1'
+  data.tar.gz: 446b92140447fdf3a45abae203308a1ac716f9ca6e6e15bd79053059c44a60fb2345a5421e7c391e855ba4739682258b5e01207ce52775e2eb6561a40a23db09

data/{LICENSE → LICENSE.txt}

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2018 Nels Nelson
+Copyright (c) 2016-2024 Nels Nelson
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/README.md

@@ -14,12 +14,13 @@ Follow these instructions to get a websocket server echo program running in your
 
 ### Container
 
-You may run the websocket server in a container.
+You may run the websocket server in a container. Using [colima] for a container runtime is recommended.
 
 ```sh
 colima start
-docker-compose up &
+docker-compose up --detach
 open http://localhost:4000/client.html
+docker-compose down
 ```
 
 
@@ -35,34 +36,33 @@ docker run --detach --publish 4000:4000 --name websocket-server-jruby websocket-
 
 Run directly with the required dependencies installed.
 
-### Install asdf
 
-The [asdf] CLI tool used to manage multiple runtime versions.
+## Install mise-en-place
+
+The [mise] CLI tool used to manage multiple runtime versions.
+
+See: https://mise.jdx.dev/getting-started.html
 
 ```sh
-git clone https://github.com/asdf-vm/asdf.git "${HOME}/.asdf" --branch v0.8.1
-pushd "${HOME}/.asdf"; git fetch origin; popd
-source "${HOME}/.asdf/asdf.sh"; source "${HOME}/.asdf/completions/asdf.bash"
+curl https://mise.jdx.dev/install.sh | sh
+~/.local/bin/mise --version
+mise 2024.x.x
 ```
 
-### Install required runtime software
-
-Download and install the latest version of the [Java JDK].
+Enable mise activation in future zsh sessions.
 
 ```sh
-asdf plugin add java
-asdf install java openjdk-17.0.2
+echo 'eval "$(~/.local/bin/mise activate zsh)"' >> ~/.zshrc
 ```
 
 
-Download and install the latest version of [JRuby].
+## Install required runtime software
+
+Use mise to install the runtime software defined as requirements
+in the .tool-versions file.
 
 ```sh
-asdf plugin add ruby
-pushd "${HOME}/.asdf/plugins/ruby/ruby-build"; git fetch origin; git pull origin master; popd
-# ~/.asdf/plugins/ruby/bin/list-all
-asdf list all ruby
-asdf install
+mise install
 ```
 
 
@@ -106,12 +106,11 @@ bundle exec rake publish
 Here is a bird's-eye view of the project layout.
 
 ```sh
-# date && tree
-Wed Apr  6 23:05:28 CDT 2022
+# date && tree -A -I "logs|vendor|tmp|Gemfile.lock"
+Sun Jul 31 15:17:32 CDT 2022
 .
 ├── Dockerfile
 ├── Gemfile
-├── Gemfile.lock
 ├── LICENSE
 ├── README.md
 ├── Rakefile
@@ -119,10 +118,9 @@ Wed Apr  6 23:05:28 CDT 2022
 ├── exe
 │   └── websocket
 ├── lib
-│   ├── log.rb
+│   ├── logging.rb
 │   ├── server
-│   │   ├── mime_types.rb
-│   │   └── server.rb
+│   │   └── mime_types.rb
 │   ├── websocket
 │   │   ├── arguments_parser.rb
 │   │   ├── channel_initializer.rb
@@ -135,22 +133,20 @@ Wed Apr  6 23:05:28 CDT 2022
 │   │   ├── http_static_file_server_handler_instance_methods.rb
 │   │   ├── idle_handler.rb
 │   │   ├── idle_state_user_event_handler.rb
+│   │   ├── instance_methods.rb
 │   │   ├── listenable.rb
 │   │   ├── message_handler.rb
-│   │   ├── modular_handler.rb
 │   │   ├── response_helpers.rb
 │   │   ├── server.rb
-│   │   ├── server_instance_methods.rb
 │   │   ├── shutdown_hook.rb
 │   │   ├── ssl_cipher_inspector.rb
 │   │   ├── ssl_context_initialization.rb
 │   │   ├── telnet_proxy.rb
 │   │   ├── validation_helpers.rb
 │   │   └── version.rb
+│   ├── websocket-server.rb
 │   ├── websocket_client.rb
 │   └── websocket_server.rb
-├── logs
-│   └── server.log
 ├── spec
 │   ├── spec_helper.rb
 │   ├── test_spec.rb
@@ -172,14 +168,17 @@ Wed Apr  6 23:05:28 CDT 2022
 │       │   ├── console.js
 │       │   └── websocket.js
 │       └── jquery.min.js
+├── websocket-server-1.0.1-java.gem
 ├── websocket-server-jruby.gemspec
-└── websocket.rb
+├── websocket.rb
+└── websocket_server.png
 
-13 directories, 52 files
+12 directories, 51 files
 ```
 
 [license]: https://gitlab.com/nelsnelson/websocket-server-jruby/blob/master/LICENSE
-[asdf]: https://asdf-vm.com/
+[mise]: https://mise.jdx.dev/
+[colima]: https://github.com/abiosoft/colima
 [Netty project]: https://github.com/netty/netty
 [Java JDK]: https://www.java.com/en/download/
 [JRuby]: https://jruby.org/download

data/exe/websocket

@@ -8,7 +8,7 @@
 
 # =begin
 #
-# Copyright Nels Nelson 2016-2022 but freely usable (see license)
+# Copyright Nels Nelson 2016-2024 but freely usable (see license)
 #
 # =end
 

data/lib/{log.rb → logging.rb}

@@ -1,6 +1,9 @@
 # encoding: utf-8
 # frozen_string_literal: false
 
+# -*- mode: ruby -*-
+# vi: set ft=ruby :
+
 # =begin
 
 # Copyright Nels Nelson 2016-2019 but freely usable (see license)
@@ -8,11 +11,10 @@
 # =end
 
 require 'java'
+require 'fileutils'
 require 'logger'
 
-require 'log4j-2'
-
-require 'fileutils'
+require 'apache-log4j-2'
 
 # The Logging module
 module Logging
@@ -22,16 +24,22 @@ module Logging
       lib_dir_path = File.expand_path(__dir__)
       project_dir_path = File.expand_path(File.dirname(lib_dir_path))
       logs_dir_path = File.expand_path(File.join(project_dir_path, 'logs'))
-      server_log_file = File.expand_path(File.join(logs_dir_path, 'server.log'))
+      FileUtils.mkdir_p(logs_dir_path)
+      log_file_path = File.expand_path(File.join(logs_dir_path, 'server.log'))
+      FileUtils.touch(log_file_path)
       {
-        level: :info,
+        level: Logger::INFO,
         name: 'websocket',
         lib_dir_path: lib_dir_path,
         project_dir_path: project_dir_path,
         logs_dir_path: logs_dir_path,
-        server_log_file: server_log_file,
-        rolling_log_file_name_template: 'server-%d{yyyy-MM-dd}.log.gz',
-        logger_pattern_template: '%d{ABSOLUTE} %-5p [%c{1}] %m%n',
+        log_file_path: log_file_path,
+        rolling_log_file_name_template: File.expand_path(
+          File.join(logs_dir_path, 'server-%d{yyyy-MM-dd}.log.gz')),
+        logging_pattern_template: {
+          java: '%d{ABSOLUTE} %-5p [%c{1}] %m%n',
+          ruby: "%<timestamp>s %-5<severity>s [%<progname>s] %<msg>s\n"
+        },
         schedule: '0 0 0 * * ?',
         size: '100M'
       }
@@ -41,69 +49,52 @@ module Logging
   # rubocop: enable Metrics/MethodLength
 end
 
-# The LogInitialization module
-module LogInitialization
-  def init
-    init_log_file
-    init_log4j if defined? Java
-  end
-  module_function :init
-
-  def init_log_file
-    FileUtils.mkdir_p(Logging.config[:logs_dir_path])
-    return if File.file?(Logging.config[:server_log_file])
-
-    File.write(Logging.config[:server_log_file], '')
+# The Logging module
+module Logging
+  if defined?(Java)
+  java_import Java::org.apache.logging.log4j.core.appender::ConsoleAppender
+  java_import Java::org.apache.logging.log4j.core.config::Configurator
+  java_import Java::org.apache.logging.log4j.core.config.builder.api::ConfigurationBuilderFactory
   end
-  module_function :init_log_file
 
   # rubocop: disable Metrics/AbcSize
   # rubocop: disable Metrics/MethodLength
   def init_log4j(log_level = org.apache.logging.log4j.Level::INFO)
-    server_log_file = Logging.config[:server_log_file]
-    logs_dir_path = Logging.config[:logs_dir_path]
-    rolling_log_file_name_template = Logging.config[:rolling_log_file_name_template]
-    rolling_log_file_path = File.join(logs_dir_path, rolling_log_file_name_template)
-
     java.lang::System.setProperty('log4j.shutdownHookEnabled', java.lang::Boolean.toString(false))
-    factory = org.apache.logging.log4j.core.config.builder.api::ConfigurationBuilderFactory
-    config = factory.newConfigurationBuilder()
+    config = ConfigurationBuilderFactory.newConfigurationBuilder()
 
-    if log_level.is_a?(Symbol)
-      log_level = org.apache.logging.log4j.Level.to_level(
-        log_level.to_s.upcase
-      )
-    end
+    log_level = org.apache.logging.log4j::Level.to_level(log_level.to_s.upcase) if log_level.is_a? Symbol
     config.setStatusLevel(log_level)
-    config.setConfigurationName(Logging.config['name'])
+    config.setConfigurationName('websocket')
 
-    # Create a console appender
-    target = org.apache.logging.log4j.core.appender::ConsoleAppender::Target::SYSTEM_OUT
+    # create a console appender
+    target = ConsoleAppender::Target::SYSTEM_OUT
+    pattern = Logging.config[:logging_pattern_template][:java]
     layout = config.newLayout('PatternLayout')
-    layout = layout.addAttribute('pattern', Logging.config[:logger_pattern_template])
+    layout = layout.addAttribute('pattern', pattern)
     appender = config.newAppender('stdout', 'CONSOLE')
     appender = appender.addAttribute('target', target)
     appender = appender.add(layout)
     config.add(appender)
 
-    # Create a root logger
+    # create a root logger
     root_logger = config.newRootLogger(log_level)
     root_logger = root_logger.add(config.newAppenderRef('stdout'))
 
-    # Create a rolling file appender
+    # create a rolling file appender
     cron = config.newComponent('CronTriggeringPolicy')
-    cron = cron.addAttribute('schedule', Logging.config[:schedule])
+    cron = cron.addAttribute('schedule', '0 0 0 * * ?')
 
     size = config.newComponent('SizeBasedTriggeringPolicy')
-    size = size.addAttribute('size', Logging.config[:size])
+    size = size.addAttribute('size', '100M')
 
     policies = config.newComponent('Policies')
     policies = policies.addComponent(cron)
     policies = policies.addComponent(size)
 
     appender = config.newAppender('rolling_file', 'RollingFile')
-    appender = appender.addAttribute('fileName', server_log_file)
-    appender = appender.addAttribute('filePattern', rolling_log_file_path)
+    appender = appender.addAttribute('fileName', Logging.config[:log_file_path])
+    appender = appender.addAttribute('filePattern', Logging.config[:rolling_log_file_name_template])
     appender = appender.add(layout)
     appender = appender.addComponent(policies)
     config.add(appender)
@@ -113,28 +104,20 @@ module LogInitialization
     config.add(root_logger)
 
     logging_configuration = config.build()
-    ctx = org.apache.logging.log4j.core.config::Configurator.initialize(logging_configuration)
+    ctx = Configurator.initialize(logging_configuration)
     ctx.updateLoggers()
   end
+  module_function :init_log4j
   # rubocop: enable Metrics/AbcSize
   # rubocop: enable Metrics/MethodLength
   # def init_log4j
-  module_function :init_log4j
-end
-# module LogInitialization
-
-::LogInitialization.init
 
-# The Apache log4j Logger class
-# rubocop: disable Style/ClassAndModuleChildren
-class org.apache.logging.log4j.core::Logger
-  alias log4j_error error
-  def error(error_or_message, error = nil)
-    return extract_backtrace(error_or_message) if error.nil?
-    log4j_error(generate_message(error_or_message, error))
-    extract_backtrace(error)
-  end
+  init_log4j if defined?(Java)
+end
+# module Logging
 
+# Namespace for methods to help with implicit backtrace printing
+module LoggerHelpers
   def generate_message(error_or_message, error)
     error_message = "#{error_or_message}: #{error.class.name}"
     error_message << ": #{error.message}" if error.respond_to?(:message)
@@ -142,40 +125,67 @@ class org.apache.logging.log4j.core::Logger
   end
 
   def extract_backtrace(error, default_result = nil)
-    log4j_error(error)
     if error.respond_to?(:backtrace)
-      error.backtrace.each { |trace| log4j_error(trace) unless trace.nil? }
+      error.backtrace.each { |trace| original_error(trace) unless trace.nil? }
     elsif error.respond_to?(:getStackTrace)
-      error.getStackTrace().each { |trace| log4j_error(trace) unless trace.nil? }
+      error.getStackTrace().each { |trace| original_error(trace) unless trace.nil? }
     else
       default_result
     end
   end
 end
-# rubocop: enable Style/ClassAndModuleChildren
+
+# Monkey-patch the built-in Ruby Logger class to support
+# implicit backtrace printing
+# TODO: Figure out if this is actually useful.
+class Logger
+  include LoggerHelpers
+
+  alias original_error error
+  def error(error_or_message, error = nil)
+    return extract_backtrace(error_or_message) if error.nil?
+    original_error(generate_message(error_or_message, error))
+    extract_backtrace(original_error(error))
+  end
+end
 
 # The Logging module
 module Logging
-  def init_logger(level = :all, logger_name = nil)
+  if defined?(Java)
+  java_import Java::org.apache.logging.log4j.Level
+  java_import Java::org.apache.logging.log4j.LogManager
+  end
+
+  FORWARD_SLASH_PATTERN = %r{/}
+
+  def init_logger(level = :info, logger_name = nil)
     return init_java_logger(level, logger_name, caller[2]) if defined?(Java)
-    init_ruby_logger(level)
+    init_ruby_logger(level, logger_name, caller[2])
   end
 
-  def init_ruby_logger(level)
-    logger_instance = Logger.new
-    logger_instance.level = Logging::Level.to_level(level.to_s.upcase)
-    logger_instance
+  def ruby_log_formatter(severity_level, datetime, program_name, message)
+    format(
+      Logging.config[:logging_pattern_template][:ruby],
+      timestamp: datetime.strftime(Logging.config[:logging_timestamp_format]),
+      progname: program_name, severity: severity_level, msg: message)
   end
 
-  # rubocop: disable Metrics/AbcSize
-  def init_java_logger(level, logger_name = nil, source_location = nil)
+  def init_ruby_logger(level = nil, logger_name = nil, source_location = nil)
     logger_name = get_formatted_logger_name(logger_name)
-    logger_name = source_location.split(/\//).last if logger_name.empty?
-    logger_instance = org.apache.logging.log4j.LogManager.getLogger(logger_name)
-    logger_instance.level = org.apache.logging.log4j.Level.to_level(level.to_s.upcase)
-    logger_instance
+    logger_name = source_location.split(FORWARD_SLASH_PATTERN).last if logger_name.empty?
+    log = Logger.new($stdout, progname: logger_name)
+    log.level = level.to_s unless level.nil?
+    log.formatter = method(:ruby_log_formatter)
+    log
+  end
+
+  def init_java_logger(level = nil, logger_name = nil, source_location = nil)
+    logger_name = get_formatted_logger_name(logger_name)
+    logger_name = source_location.split(FORWARD_SLASH_PATTERN).last if logger_name.empty?
+    log = LogManager.getLogger(logger_name)
+    log.level = Level.to_level(level.to_s.upcase) unless level.nil?
+    log
   end
-  # rubocop: enable Metrics/AbcSize
 
   def get_formatted_logger_name(logger_name = nil)
     return logger_name.to_s[/\w+$/] unless logger_name.nil?
@@ -202,14 +212,13 @@ module Logging
     when 1 then :info
     when 0 then :debug
     when -1 then :trace
-    when -2..-Float::INFINITY then :all
+    when -Float::INFINITY..-2 then :all
     end
   end
   # rubocop: enable Metrics/CyclomaticComplexity
-  module_function :symbolize_numeric_log_level
 
-  def log_level=(log_level)
-    Logging.config[:level] = symbolize_numeric_log_level(log_level)
+  def log_level=(level)
+    Logging.config[:level] = symbolize_numeric_log_level(level)
   end
   module_function :log_level=
 
@@ -218,7 +227,7 @@ module Logging
   end
   module_function :log_level
 
-  def log(level = Logging.log_level, log_name = nil)
+  def log(level = Logging.log_level, log_name = Logging.config[:app_name])
     @log ||= init_logger(level, log_name)
   end
   alias logger log

data/lib/server/mime_types.rb

@@ -6,7 +6,7 @@
 
 # =begin
 #
-# Copyright Nels Nelson 2016-2022 but freely usable (see license)
+# Copyright Nels Nelson 2016-2024 but freely usable (see license)
 #
 # =end
 

data/lib/websocket/arguments_parser.rb

@@ -6,7 +6,7 @@
 
 # =begin
 #
-# Copyright Nels Nelson 2016-2022 but freely usable (see license)
+# Copyright Nels Nelson 2016-2024 but freely usable (see license)
 #
 # =end
 
@@ -16,20 +16,21 @@ require_relative 'config'
 
 # The WebSocket module
 module WebSocket
+  # rubocop: disable Metrics/ClassLength
   # The ArgumentsParser class
   class ArgumentsParser
-    Flags = %i[
-      banner port telnet_proxy_host telnet_proxy_port
-      ssl ssl_certificate ssl_private_key use_jdk_ssl_provider
-      inspect_ssl idle_reading idle_writing log_requests web_root
-      verbose help version
-    ].freeze
     attr_reader :parser, :options
 
     def initialize(option_parser = OptionParser.new)
       @parser = option_parser
       @options = ::WebSocket.server_config.dup
-      Flags.each { |method_name| method(method_name).call }
+      @flags = %i[
+        banner port telnet_proxy_host telnet_proxy_port
+        ssl ssl_certificate ssl_private_key use_jdk_ssl_provider
+        inspect_ssl idle_reading idle_writing log_requests web_root
+        verbose help version
+      ]
+      @flags.each { |method_name| method(method_name)&.call if respond_to?(method_name) }
     end
 
     def banner
@@ -38,9 +39,17 @@ module WebSocket
       @parser.separator 'Options:'
     end
 
+    def validated_port(val, integer_pattern = /^\d+$/)
+      raise OptionParser::InvalidArgument, "Invalid port: #{v}" unless \
+        integer_pattern.match?(val.to_s) && val.positive? && val < 65_536
+
+      val
+    end
+
     def port
-      @parser.on_head('-p', '--port=port', 'Listen on this port for incoming connections') do |v|
-        @options[:port] = v.to_i
+      description = "Port on which to listen for connections; default: #{@options[:port]}"
+      @parser.on('-p', '--port=<port>', Integer, description) do |v|
+        @options[:port] = validated_port(v).to_i
       end
     end
 
@@ -112,6 +121,7 @@ module WebSocket
 
     def verbose
       @parser.on_tail('-v', '--verbose', 'Increase verbosity') do
+        @options[:log_level] ||= 0
         @options[:log_level] -= 1
       end
     end
@@ -125,17 +135,23 @@ module WebSocket
 
     def version
       @parser.on_tail('--version', 'Show version') do
-        puts "#{File.basename($PROGRAM_NAME)} version #{WebSocket.version}"
+        puts "#{File.basename($PROGRAM_NAME)} version #{::WebSocket::VERSION}"
         exit
       end
     end
   end
+  # rubocop: enable Metrics/ClassLength
   # class ArgumentsParser
 
   def parse_arguments(arguments_parser = WebSocket::ArgumentsParser.new)
     arguments_parser.parser.parse!(ARGV)
     arguments_parser.options
-  rescue OptionParser::InvalidOption, OptionParser::AmbiguousOption => e
+  rescue OptionParser::InvalidArgument, OptionParser::InvalidOption,
+         OptionParser::MissingArgument, OptionParser::NeedlessArgument => e
+    puts e.message
+    puts parser
+    exit
+  rescue OptionParser::AmbiguousOption => e
     abort e.message
   end
 end

data/lib/websocket/channel_initializer.rb

@@ -6,7 +6,7 @@
 
 # =begin
 #
-# Copyright Nels Nelson 2016-2022 but freely usable (see license)
+# Copyright Nels Nelson 2016-2024 but freely usable (see license)
 #
 # =end
 

data/lib/websocket/config.rb

@@ -6,7 +6,7 @@
 
 # =begin
 #
-# Copyright Nels Nelson 2016-2022 but freely usable (see license)
+# Copyright Nels Nelson 2016-2024 but freely usable (see license)
 #
 # =end
 

data/lib/websocket/encoding.rb

@@ -6,7 +6,7 @@
 
 # =begin
 #
-# Copyright Nels Nelson 2016-2022 but freely usable (see license)
+# Copyright Nels Nelson 2016-2024 but freely usable (see license)
 #
 # =end
 

data/lib/websocket/file_server_channel_progressive_future_listener.rb

@@ -6,7 +6,7 @@
 
 # =begin
 #
-# Copyright Nels Nelson 2016-2022 but freely usable (see license)
+# Copyright Nels Nelson 2016-2024 but freely usable (see license)
 #
 # =end
 

data/lib/websocket/frame_handler.rb

@@ -6,7 +6,7 @@
 
 # =begin
 #
-# Copyright Nels Nelson 2016-2022 but freely usable (see license)
+# Copyright Nels Nelson 2016-2024 but freely usable (see license)
 #
 # =end
 

data/lib/websocket/header_helpers.rb

@@ -6,7 +6,7 @@
 
 # =begin
 #
-# Copyright Nels Nelson 2016-2022 but freely usable (see license)
+# Copyright Nels Nelson 2016-2024 but freely usable (see license)
 #
 # =end
 

data/lib/websocket/http_static_file_server_handler.rb

@@ -6,7 +6,7 @@
 
 # =begin
 #
-# Copyright Nels Nelson 2016-2022 but freely usable (see license)
+# Copyright Nels Nelson 2016-2024 but freely usable (see license)
 #
 # =end
 

data/lib/websocket/http_static_file_server_handler_instance_methods.rb

@@ -6,7 +6,7 @@
 
 # =begin
 #
-# Copyright Nels Nelson 2016-2022 but freely usable (see license)
+# Copyright Nels Nelson 2016-2024 but freely usable (see license)
 #
 # =end
 
@@ -36,8 +36,8 @@ module WebSocket
 
   # The HttpStaticFileServerHandlerInstanceMethods module
   module HttpStaticFileServerHandlerInstanceMethods
-    ForwardSlashBeforeEOLPattern = %r{/$}.freeze
-    URIForwardSlashTemplate = '%<uri>s/'.freeze
+    FORWARD_SLASH_BEFORE_EOL_PATTERN = %r{/$}
+    URI_FORWARD_SLASH_TEMPLATE = '%<uri>s/'.freeze
 
     # rubocop: disable Metrics/AbcSize
     # rubocop: disable Metrics/CyclomaticComplexity
@@ -69,10 +69,10 @@ module WebSocket
       end
 
       if File.directory? path
-        if ForwardSlashBeforeEOLPattern.match?(uri)
+        if FORWARD_SLASH_BEFORE_EOL_PATTERN.match?(uri)
           send_listing(ctx, path)
         else
-          send_redirect(ctx, format(URIForwardSlashTemplate, uri: uri))
+          send_redirect(ctx, format(URI_FORWARD_SLASH_TEMPLATE, uri: uri))
         end
         return
       end
@@ -149,7 +149,6 @@ module WebSocket
     # rubocop: enable Metrics/PerceivedComplexity
 
     def exceptionCaught(ctx, cause)
-      log.info "##{__method__} wtf2"
       cause.printStackTrace()
       return unless ctx.channel().isActive()
       send_error(ctx, HttpResponseStatus::INTERNAL_SERVER_ERROR)