intranet-core 2.3.1 → 2.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: db45417345875ffcbd1500ce99febcb8e52cdd55934c44812310e75d8dbd85a1
-  data.tar.gz: e330a7b69c123a4c380682dbf3a39ff3a92365f8d0bdcf4ceaa1e175a74ad497
+  metadata.gz: e9d5ce66790b8b308a7f4d366aaa0a24a4d8e4c8ea82d31c6a5b78dd93fb2a49
+  data.tar.gz: eaff940db100fbebae0ebc6fd7c204ce6a33117e7b2c779a5f4961da162a230f
 SHA512:
-  metadata.gz: 360437f3351b25f886a44da9232ab7d844f4fcfcc25d1030b92ea3f6b24370613a3bc9a0dc3fc80ac000df6365a4cdc39da4c85f50ed3351603a345797e1e842
-  data.tar.gz: ad870fc6849c997b62d60618828c292087d3a0232202d340bd18772dda6241f11ccd86a6bd91589f66f3e1cbd67165356728a0e552baf0019f30bc207c10a1d2
+  metadata.gz: b48372ac7f9d356b60685ac561aee1d3b3a596c188493082a8107194c0d08f9230b654796057a83a2b463138baee381045263971f7fc063ff20ffcfcf3b91c5d
+  data.tar.gz: 0f6ea7adb342e03243ef249a17773f7bf4db6593a09386888451948103b7730e4cc0244173bd397befb4960301d9f3a33cffc00839b273444538f714af927f31

data/lib/intranet/abstract_responder.rb

@@ -37,30 +37,44 @@ module Intranet
       # nothing to do
     end
 
-    # Generates the HTML content associated to the given +path+ and +query+.
+    # Generates the HTML answer (HTTP return code, MIME type and body) associated to the given
+    # +path+ and +query+.
+    #
+    # The function may return a partial content, in which case the HTTP return code must be 206 and
+    # the answer body is expected to be a +Hash+ with the following keys:
+    # * +:title+ (mandatory): the page title
+    # * +:content+ (mandatory): the partial content
+    # * +:stylesheets+ (optional): an array of the required Cascade Style Sheets (CSS) files, either
+    #   absolute or relative to the module root
+    # * +:scripts+ (optional): an array of hashes, each representing a <script> element to be
+    #   included in the generated page. Keys are the one accepted by the HTML <script> tag. The
+    #   +:src+ key may be either absolute or relative to the module root
     # @param path [String] The requested URI, relative to that module root URI
     # @param query [Hash] The URI variable/value pairs, if any
-    # @return [Array] The HTTP return code, the MIME type and the answer body.
+    # @return [Array<Integer, String, String> or Array<Integer, String, Hash>] The HTTP return code,
+    #         the MIME type and the answer body (partial if return code is 206).
     def generate_page(path, query)
       [404, '', '']
     end
 
     # Provides the list of Cascade Style Sheets (CSS) dependencies for this module, either using
     # absolute or relative (from the module root) paths.
-    # If redefined, this method should probably append dependencies rather than overwriting them.
+    # @deprecated Use {generate_page} partial content feature, setting +:stylesheets+ attribute of
+    #             the returned body.
     # @return [Array] The list of CSS dependencies, as absolute path or relative to the module root
     #                 URL.
     def css_dependencies
-      ['/design/style.css']
+      []
     end
 
     # Provides the list of Javascript files (JS) dependencies for this module, either using
     # absolute or relative (from the module root) paths.
-    # If redefined, this method should probably append dependencies rather than overwriting them.
+    # @deprecated Use {generate_page} partial content feature, setting +:scripts+ attribute of the
+    #             returned body.
     # @return [Array] The list of JS dependencies, as absolute path or relative to the module root
     #                 URL.
     def js_dependencies
-      ['/design/nav.js']
+      []
     end
   end
 end

data/lib/intranet/core/builder.rb

@@ -117,8 +117,9 @@ module Intranet
       # @param responder [Intranet::AbstractResponder] The responder that produced the body
       # @param path [String] The path to the responder
       def add_header_and_footer(body, responder, path)
-        to_markup('skeleton', body: body, css: responder.css_dependencies,
-                              js: responder.js_dependencies, current_path: path)
+        body[:stylesheets] ||= responder.css_dependencies
+        body[:scripts] ||= responder.js_dependencies.map { |url| { src: url, defer: 'defer' } }
+        to_markup('skeleton', body: body, current_path: path)
       end
 
       # Check whether a path is valid. In particular, this function excludes a path containing an

data/lib/intranet/core/version.rb

@@ -6,7 +6,7 @@ module Intranet
     NAME = 'intranet-core'
 
     # The version of the gem, according to semantic versionning.
-    VERSION = '2.3.1'
+    VERSION = '2.4.1'
 
     # The URL of the gem homepage.
     HOMEPAGE_URL = 'https://rubygems.org/gems/intranet-core'

data/lib/intranet/core.rb

@@ -91,10 +91,11 @@ module Intranet
     # See https://github.com/nahi/webrick/blob/master/lib/webrick/accesslog.rb#L69
     ACCESSLOG_FMT = "%h '%r' -> %s (%b bytes in %Ts)"
 
-    def load_http_server(preferred_port)
+    def load_http_server(preferred_port) # rubocop:disable Metrics/MethodLength
       @port = preferred_port
       begin
-        WEBrick::HTTPServer.new(Port: @port, Logger: @logger, AccessLog: [[@logger, ACCESSLOG_FMT]])
+        WEBrick::HTTPServer.new(Port: @port, Logger: @logger, AccessLog: [[@logger, ACCESSLOG_FMT]],
+                                ServerSoftware: nil)
       rescue Errno::EACCES # not enough permission to use port 80
         @port = 8080
         retry

data/lib/intranet/resources/haml/skeleton.haml

@@ -3,24 +3,23 @@
   %head
     %title= Socket.gethostname.capitalize + ' | ' + body[:title]
     %meta{charset: 'utf-8'}
+    %meta{name: 'viewport', content: 'width=device-width, initial-scale=1.0'}
     %link{rel: 'icon', type: 'image/x-icon', href: '/design/favicon.ico'}
-    - css.each do |url|
-      - if url.start_with?('/')
-        %link{rel: 'stylesheet', type: 'text/css', href: url}
-      - else
-        %link{rel: 'stylesheet', type: 'text/css', href: current_path + '/' + url}
-    - js.each do |url|
-      - if url.start_with?('/')
-        %script{src: url, defer: 'defer'}
-      - else
-        %script{src: current_path + '/' + url, defer: 'defer'}
+    %link{rel: 'stylesheet', type: 'text/css', href: '/design/style.css'}
+    - body[:stylesheets].each do |url|
+      - url = current_path + '/' + url unless url.start_with?('/')
+      %link{rel: 'stylesheet', type: 'text/css', href: url}
+    %script{type: 'module', src: '/design/nav.js'}
+    - body[:scripts].each do |script|
+      - script[:src] = current_path + '/' + script[:src] unless script[:src].start_with?('/')
+      %script{script}
   %body
     %header
-      %a{id: 'openmenu', onclick: 'openNavMenu();'}= '☰'
+      %a{id: 'openmenu'}= '☰'
       %h1
         %a{href: '/index.html', title: I18n.t('nav.back.home')}= Socket.gethostname.capitalize
       %nav
-        %a{id: 'closemenu', onclick: 'closeNavMenu();'}= '×'
+        %a{id: 'closemenu'}= '×'
         %ul
           - responders.children_nodes.each do |path, node|
             - if node.children?
@@ -48,7 +47,7 @@
         = ' ' + I18n.t('nav.generated.on_date') + ' ' + Time.now.strftime(I18n.t('date_format'))
         = ' ' + I18n.t('date_time_separator') + Time.now.strftime(I18n.t('time_format'))
         %br
-        %a{onclick: 'openModal();'}= I18n.t('nav.about')
+        %a{id: 'openmodal'}= I18n.t('nav.about')
       %aside#modal
         %div#modal-content
           %h2= 'Intranet'

data/lib/intranet/resources/www/nav.js

@@ -4,23 +4,13 @@
  * and for the modal box.
  */
 
+const openMenu = document.getElementById('openmenu');
+const closeMenu = document.getElementById('closemenu');
+const navMenu = document.querySelectorAll('header nav')[0];
+openMenu.addEventListener('click', function() { navMenu.style.width = 'auto'; });
+closeMenu.addEventListener('click', function() { navMenu.style.width = ''; });
 
-function openNavMenu() {
-  document.querySelectorAll('header nav')[0].style.width = 'auto';
-}
-
-function closeNavMenu() {
-  /* Remove value property set in openNavMenu() */
-  document.querySelectorAll('header nav')[0].style.width = '';
-}
-
-function openModal() {
-  document.getElementById('modal').style.display = 'block';
-}
-
-window.onclick = function(event) {
-  if (event.target == document.getElementById('modal')) {
-    document.getElementById('modal').style.display = 'none';
-  }
-}
-
+const openModal = document.getElementById('openmodal');
+const modal = document.getElementById('modal');
+openModal.addEventListener('click', function() { modal.style.display = 'block'; });
+modal.addEventListener('click', function() { modal.style.display = 'none'; });

data/lib/intranet/resources/www/style.css

@@ -24,12 +24,11 @@ body {
     background: #1e262b;
     font-family: "Source Sans Pro", Cantarell, sans-serif;
     color: black;
-}
-/* Mobile devices only */
-@media only screen and (max-width: 600px), only screen and (max-device-width: 600px) {
-    body {
-        font-size: 180%;
-    }
+    font-size: 1.0rem;
+    -moz-text-size-adjust: none;
+    -webkit-text-size-adjust: none;
+    -ms-text-size-adjust: none;
+    font-size-adjust: none;
 }
 
 a {
@@ -51,7 +50,7 @@ header {
 }
 header h1 {
     float: left;
-    font-size: 225%;
+    font-size: 2.25em;
     font-weight: 500;
     margin: 0px;
     padding: 5px 20px 8px; /* top sides bottom */
@@ -67,11 +66,11 @@ header nav ul {
 header nav > ul > li {
     display: inline-block;
 }
-header nav > ul > li a, header a#openmenu, header a#closemenu {
+header nav > ul > li a {
     display: block;
     padding: 16px 20px;
     margin: 0px;
-    font-size: 125%;
+    font-size: 1.25em;
     color: rgba(255, 255, 255, 0.7);
 }
 header nav > ul > li a:hover {
@@ -100,21 +99,18 @@ header a#openmenu, header a#closemenu {
 
 /* Mobile devices only */
 @media only screen and (max-width: 600px), only screen and (max-device-width: 600px) {
-    header {
-        height: 100px;
-    }
     header a#openmenu {
         display: block;
         position: absolute;
-        top: 0;
-        left: 0;
-        font-size: 190%;
+        top: 12%;
+        left: 20px;
+        font-size: 2em;
         font-weight: bold;
         color: white;
     }
     header nav a#closemenu {
         display: block;
-        font-size: 250%;
+        font-size: 2.5em;
         font-weight: bold;
         color: white;
         text-align: right;
@@ -132,21 +128,18 @@ header a#openmenu, header a#closemenu {
         padding: 0;
         z-index: 1;
         background: inherit;
-        font-size: 125%;
+        font-size: 1.25em;
         overflow-y: scroll;
         width: 0;
         max-width: 100%;
     }
     header nav > ul {
         height: auto;
-        margin: 0px 0px 50px; /* top sides bottom */
     }
     header nav > ul > li {
         display: block;
     }
     header nav > ul > li a {
-        color: rgba(255, 255, 255, 1.0);
-        margin-top: 35px;
         padding: 10px 150px 10px 40px; /* top right bottom left */
     }
     header nav > ul > li > ul {
@@ -154,21 +147,16 @@ header a#openmenu, header a#closemenu {
         position: static; /* reset absolute positionning */
     }
     header nav > ul > li > ul > li a {
-        color: rgba(255, 255, 255, 0.7);
         margin: 0px;
         padding: 10px 100px 10px 90px; /* top right bottom left */
     }
-    header nav li a:hover {
-        color: inherit;
-        background-color: #c2c2c2;
-    }
 }
 
-/* breadcrump navigation menu */
+/* breadcrumb navigation menu */
 ul.breadcrumb {
     list-style: none;
     padding: 0px;
-    font-size: 90%;
+    font-size: 0.9em;
     margin-bottom: 50px;
 }
 ul.breadcrumb li {
@@ -178,7 +166,7 @@ ul.breadcrumb li+li:before {
     padding: 4px;
     content: "/\00a0";
 }
-ul.breadcrumb li a, main article a {
+main a {
     color: #748ea3;
 }
 
@@ -193,7 +181,7 @@ ul.breadcrumb li a, main article a {
 
 body > main {
     background: #f7f8fa;
-    padding: 57px 0px; /* height of the navigation bar */
+    padding: 58px 0px; /* height of the navigation bar */
 }
 
 body > main section, body > main article, body > main hr, body > footer p {
@@ -211,16 +199,13 @@ body > main hr {
 
 /* Mobile devices only */
 @media only screen and (max-width: 600px), only screen and (max-device-width: 600px) {
-    body > main {
-        padding: 100px 0px;
-    }
-    body > main section, main hr, footer p {
+    body > main section, body > main hr, body > footer p {
         width: 90%;
     }
 }
 
 body > main section h2 {
-    font-size: 200%;
+    font-size: 2em;
     text-align: center;
     margin: 50px 0px 2em; /* top sides bottom */
 }
@@ -228,6 +213,7 @@ body > main article h2 {
     margin-bottom: 40px;
 }
 body > main section h3 {
+    font-size: 1.17em;
     margin: 3em 0px 1em; /* top sides bottom */
 }
 
@@ -250,7 +236,9 @@ body > footer {
     text-align: center;
     color: white;
     padding: 15px 0px; /* top sides */
-    font-size: 10pt;
+}
+body > footer p {
+    font-size: 0.85em;
 }
 body > footer aside#modal { /* modal box container */
     display: none;
@@ -266,11 +254,12 @@ body > footer aside#modal { /* modal box container */
 }
 body > footer aside#modal #modal-content { /* modal box */
     background-color: #f7f8fa;
-    margin: 15% auto;
+    margin: auto;
+    margin-top: 50vh;
+    transform: translateY(-50%);
     padding: 20px 25px 40px; /* top sides bottom */
     border: 1px solid #1e262b;
-    width: 400px;
-    font-size: 125%;
+    width: 425px;
 }
 body > footer aside#modal #modal-content dl {
   display: grid;
@@ -285,12 +274,6 @@ body > footer aside#modal #modal-content dl dd {
   text-align: left;
   margin-left: 0;
 }
-/* Mobile devices only */
-@media only screen and (max-width: 600px), only screen and (max-device-width: 600px) {
-    body > footer {
-        font-size: 13pt;
-    }
-}
 
 
 /******************************************* Error pages ******************************************/
@@ -308,4 +291,3 @@ section#error p img {
 section#error a {
     color: #748ea3;
 }
-

data/spec/intranet/core_spec.rb

@@ -61,6 +61,31 @@ RSpec.describe Intranet::Core do
       thread&.join
     end
 
+    it 'should not advertise server version in HTTP response headers' do
+      @intranet = described_class.new(Intranet::Logger.new(Intranet::Logger::FATAL))
+      thread = Thread.new { @intranet.start }
+      while @intranet.instance_variable_get(:@server).status != :Running
+      end
+
+      socket = TCPSocket.new('localhost', @intranet.port)
+      socket.puts("GET /design/favicon.ico HTTP/1.1\r\nHost: localhost:#{@intranet.port}\r\n\r\n")
+      server_identification = ''
+      expect(socket.gets).to include('HTTP/1.1 200 OK') # pre-requisite
+      while (line = socket.gets.chomp) # look for Server response header
+        break if line.empty?
+
+        if line.start_with?('Server: ')
+          server_identification = line[8..-1]
+          break
+        end
+      end
+      expect(server_identification).to be_empty
+      socket.close
+
+      @intranet.stop
+      thread&.join
+    end
+
     context 'when no module is registered' do
       it 'should return HTTP error 404 when requested for /index.html' do
         @intranet = described_class.new(Intranet::Logger.new(Intranet::Logger::FATAL))
@@ -219,13 +244,21 @@ RSpec.describe Intranet::Core do
       it 'should be called to retrieve the body of the page' do
         @intranet = described_class.new(Intranet::Logger.new(Intranet::Logger::FATAL))
 
-        responder = Intranet::TestResponder.new(
-          {
-            '/index.html' => [206, 'text/html', { content: 'PARTIAL_CONTENT', title: 'MyTitle' }]
-          },
-          ['/responder.css', 'nav.css'],
-          ['module.js', '/js/interactive.js']
-        )
+        responder = Intranet::TestResponder.new({
+                                                  '/index.html' => [
+                                                    206,
+                                                    'text/html',
+                                                    {
+                                                      content: 'PARTIAL_CONTENT',
+                                                      title: 'MyTitle',
+                                                      stylesheets: ['/resp.css', 'nav.css'],
+                                                      scripts: [
+                                                        { src: 'module.js', type: 'module' },
+                                                        { src: '/js/interactive.js', defer: 'defer' }
+                                                      ]
+                                                    }
+                                                  ]
+                                                })
         @intranet.register_module(responder, ['r'], responder.resources_dir)
 
         thread = Thread.new { @intranet.start }
@@ -255,11 +288,11 @@ RSpec.describe Intranet::Core do
         expect(html).to match(%r{<footer>.*#{hostname}.*</footer>}m)
 
         # Returned HTML document: includes all CSS dependencies, relative or absolute path
-        expect(html).to match(%r{<link href='/responder.css' rel='stylesheet' type='text/css'})
+        expect(html).to match(%r{<link href='/resp.css' rel='stylesheet' type='text/css'})
         expect(html).to match(%r{<link href='/r/nav.css' rel='stylesheet' type='text/css'})
 
         # Returned HTML document: includes all JS dependencies
-        expect(html).to match(%r{<script defer='defer' src='/r/module.js'></script>})
+        expect(html).to match(%r{<script src='/r/module.js' type='module'></script>})
         expect(html).to match(%r{<script defer='defer' src='/js/interactive.js'></script>})
 
         # Returned HTML document: includes Intranet Core name, version and URL
@@ -281,7 +314,7 @@ RSpec.describe Intranet::Core do
         responder = Intranet::TestResponder.new(
           '/index.html' => [206, 'text/html', { content: 'PARTIAL_CONTENT', title: 'MyTitle' }]
         )
-        other_responder = Intranet::TestResponder.new({}, [], [], true)
+        other_responder = Intranet::TestResponder.new({}, true)
         @intranet.register_module(responder, %w[r], responder.resources_dir)
         @intranet.register_module(responder, %w[dep_th1], responder.resources_dir)
         @intranet.register_module(responder, %w[depth2 res_p1], responder.resources_dir)

data/spec/test_responder/responder.rb

@@ -6,10 +6,8 @@ module Intranet
   class TestResponder < AbstractResponder
     attr_reader :finalized
 
-    def initialize(responses = {}, extra_css = [], extra_js = [], hide_from_menu = false) # rubocop:disable Metrics/ParameterLists
+    def initialize(responses = {}, hide_from_menu = false)
       @responses = responses
-      @extra_css = extra_css
-      @extra_js = extra_js
       @finalized = false
       @hide_from_menu = hide_from_menu
     end
@@ -47,14 +45,6 @@ module Intranet
       super(path, query)
     end
 
-    def css_dependencies
-      super + @extra_css
-    end
-
-    def js_dependencies
-      super + @extra_js
-    end
-
     private
 
     def dump_with_encoding(path, query)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: intranet-core
 version: !ruby/object:Gem::Version
-  version: 2.3.1
+  version: 2.4.1
 platform: ruby
 authors:
 - Ebling Mis
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-03-08 00:00:00.000000000 Z
+date: 2021-12-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: haml
@@ -211,8 +211,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6.2
+rubygems_version: 3.2.5
 signing_key: 
 specification_version: 4
 summary: Core component to build a custom intranet.