sqlui 0.1.60 → 0.1.62

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 45285cfe6706fdfa5bae7abf73f1a287e0238709f2f83a18eda8386c71f4de63
-  data.tar.gz: 9d3eacf4a4306697c4c34d91b51303dd88b376bec29526124d18ec5c4fbe43d7
+  metadata.gz: 3ba6df74cbbe88d22279ec0a6a7cc9bf1ed191d695b84fcf424ee9b03a6b64cb
+  data.tar.gz: a9f53957b16860755a7d6d21980e573af93054bff997a205d791553b8b8dae59
 SHA512:
-  metadata.gz: f94e45bf232683013994f63a46eb18257aeaf9d81dd906ccffdbdced023be1743ebca28ff87f34a3b45932757b80516fae979d74a9ab1002affa78a7adb412dc
-  data.tar.gz: 4151e2f3f7d795931abfee848f040e9b6066c4ef4ec9503d0f3cdb586e82d3ced59f50c3eb8744c2adc2847324d275d642e313c0ba825768e452e7a0f8af7b75
+  metadata.gz: d9375a77c825b5036079d8e3d4b5df74e4370be1966441bc3a8a4a887e878fd58d79500187116c2492e3d27358084c0aaa70bd45f48aba5e387ac37c0d51e678
+  data.tar.gz: ac699985a8459edb63c8aeeca34c861471d468b9680fd767d8c69f14610c27d04d55feeaa07d2c0256fa0ee6148e4f659f9060b399847173fc4cbe67d3c9b6ea

data/.release-version

@@ -1 +1 @@
-0.1.60
+0.1.62

data/app/database_config.rb

@@ -14,8 +14,8 @@ class DatabaseConfig
     @display_name = Args.fetch_non_empty_string(hash, :display_name).strip
     @description = Args.fetch_non_empty_string(hash, :description).strip
     @url_path = Args.fetch_non_empty_string(hash, :url_path).strip
-    raise ArgumentError, 'url_path should start with a /' unless @url_path.start_with?('/')
-    raise ArgumentError, 'url_path should not end with a /' if @url_path.length > 1 && @url_path.end_with?('/')
+    raise ArgumentError, 'url_path should not start with a /' if @url_path.start_with?('/')
+    raise ArgumentError, 'url_path should not end with a /' if @url_path.end_with?('/')
 
     @saved_path = Args.fetch_non_empty_string(hash, :saved_path).strip
 

data/app/server.rb

@@ -25,6 +25,9 @@ class Server < Sinatra::Base
   def self.init_and_run(config, resources_dir)
     logger.info("Starting SQLUI v#{Version::SQLUI}")
     logger.info("Airbrake enabled: #{config.airbrake[:server]&.[](:enabled) || false}")
+
+    WEBrick::HTTPRequest.const_set('MAX_URI_LENGTH', 2 * 1024 * 1024)
+
     if config.airbrake[:server]&.[](:enabled)
       require 'airbrake'
       require 'airbrake/rack'
@@ -59,51 +62,53 @@ class Server < Sinatra::Base
     set :raise_errors,    false
     set :show_exceptions, false
 
-    favicon_hash = Digest::MD5.hexdigest(File.read(File.join(resources_dir, 'favicon.svg')))
-    css_hash = Digest::MD5.hexdigest(File.read(File.join(resources_dir, 'sqlui.css')))
-    js_hash = Digest::MD5.hexdigest(File.read(File.join(resources_dir, 'sqlui.js')))
-
     get '/-/health' do
       status 200
       body 'OK'
     end
 
     get '/?' do
-      redirect config.list_url_path, 301
+      redirect config.base_url_path, 301
     end
 
-    get '/favicon.svg' do
-      headers 'Cache-Control' => 'max-age=31536000'
-      send_file File.join(resources_dir, 'favicon.svg')
+    resource_path_map = {}
+    Dir.glob(File.join(resources_dir, '*')).each do |file|
+      hash = Digest::MD5.hexdigest(File.read(file))
+      basename = File.basename(file)
+      url_path = "#{config.base_url_path}/#{basename}"
+      case File.extname(basename)
+      when '.svg'
+        content_type = 'image/svg+xml; charset=utf-8'
+      when '.css'
+        content_type = 'text/css; charset=utf-8'
+      when '.js'
+        content_type = 'text/javascript; charset=utf-8'
+      else
+        raise "unsupported resource file extension: #{File.extname(basename)}"
+      end
+      resource_path_map[basename] = "#{url_path}?#{hash}"
+      get url_path do
+        headers 'Content-Type' => content_type
+        headers 'Cache-Control' => 'max-age=31536000'
+        send_file file
+      end
     end
 
-    get "#{config.list_url_path}/?" do
+    get "#{config.base_url_path}/?" do
       headers 'Cache-Control' => 'no-cache'
-      erb :databases, locals: { config: config }
+      erb :databases, locals: { config: config, resource_path_map: resource_path_map }
     end
 
     config.database_configs.each do |database|
-      get "#{database.url_path}/?" do
+      get "#{config.base_url_path}/#{database.url_path}/?" do
         redirect "#{database.url_path}/query", 301
       end
 
-      get "#{database.url_path}/sqlui.css" do
-        headers 'Content-Type' => 'text/css; charset=utf-8'
-        headers 'Cache-Control' => 'max-age=31536000'
-        send_file File.join(resources_dir, 'sqlui.css')
-      end
-
-      get "#{database.url_path}/sqlui.js" do
-        headers 'Content-Type' => 'text/javascript; charset=utf-8'
-        headers 'Cache-Control' => 'max-age=31536000'
-        send_file File.join(resources_dir, 'sqlui.js')
-      end
-
-      post "#{database.url_path}/metadata" do
+      post "#{config.base_url_path}/#{database.url_path}/metadata" do
         metadata = database.with_client do |client|
           {
             server: "#{config.name} - #{database.display_name}",
-            list_url_path: config.list_url_path,
+            base_url_path: config.base_url_path,
             schemas: DatabaseMetadata.lookup(client, database),
             tables: database.tables,
             columns: database.columns,
@@ -131,7 +136,7 @@ class Server < Sinatra::Base
         body metadata.to_json
       end
 
-      post "#{database.url_path}/query" do
+      post "#{config.base_url_path}/#{database.url_path}/query" do
         data = request.body.read
         request.body.rewind # since Airbrake will read the body on error
         params.merge!(JSON.parse(data, symbolize_names: true))
@@ -215,7 +220,7 @@ class Server < Sinatra::Base
         end
       end
 
-      get "#{database.url_path}/download_csv" do
+      get "#{config.base_url_path}/#{database.url_path}/download_csv" do
         break client_error('missing sql') unless params[:sql]
 
         sql = Base64.decode64(params[:sql]).force_encoding('UTF-8')
@@ -250,7 +255,7 @@ class Server < Sinatra::Base
         end
       end
 
-      get(%r{#{Regexp.escape(database.url_path)}/(query|graph|structure|saved)}) do
+      get(/#{Regexp.escape("#{config.base_url_path}/#{database.url_path}/")}(query|graph|structure|saved)/) do
         status 200
         headers 'Cache-Control' => 'no-cache'
         client_config = config.airbrake[:client] || {}
@@ -259,9 +264,7 @@ class Server < Sinatra::Base
           airbrake_enabled: client_config[:enabled] || false,
           airbrake_project_id: client_config[:project_id] || '',
           airbrake_project_key: client_config[:project_key] || '',
-          js_hash: js_hash,
-          css_hash: css_hash,
-          favicon_hash: favicon_hash
+          resource_path_map: resource_path_map
         }
       end
     end
@@ -276,7 +279,12 @@ class Server < Sinatra::Base
         body json
       else
         message = "#{status} #{Rack::Utils::HTTP_STATUS_CODES[status]}"
-        erb :error, locals: { title: "SQLUI #{message}", message: message, stacktrace: stacktrace }
+        erb :error, locals: {
+          resource_path_map: resource_path_map,
+          title: "SQLUI #{message}",
+          message: message,
+          stacktrace: stacktrace
+        }
       end
     end
 

data/app/sqlui_config.rb

@@ -8,7 +8,7 @@ require_relative 'deep'
 
 # App config including database configs.
 class SqluiConfig
-  attr_reader :name, :port, :environment, :list_url_path, :database_configs, :airbrake
+  attr_reader :name, :port, :environment, :base_url_path, :database_configs, :airbrake
 
   def initialize(filename, overrides = {})
     config = YAML.safe_load(ERB.new(File.read(filename)).result, aliases: true).deep_merge!(overrides)
@@ -19,10 +19,10 @@ class SqluiConfig
     @name = Args.fetch_non_empty_string(config, :name).strip
     @port = Args.fetch_non_empty_int(config, :port)
     @environment = Args.fetch_non_empty_string(config, :environment).strip
-    @list_url_path = Args.fetch_non_empty_string(config, :list_url_path).strip
-    raise ArgumentError, 'list_url_path should start with a /' unless @list_url_path.start_with?('/')
-    if @list_url_path.length > 1 && @list_url_path.end_with?('/')
-      raise ArgumentError, 'list_url_path should not end with a /'
+    @base_url_path = Args.fetch_non_empty_string(config, :base_url_path).strip
+    raise ArgumentError, 'base_url_path should start with a /' unless @base_url_path.start_with?('/')
+    if @base_url_path.length > 1 && @base_url_path.end_with?('/')
+      raise ArgumentError, 'base_url_path should not end with a /'
     end
 
     databases = Args.fetch_non_empty_hash(config, :databases)

data/app/views/databases.erb

@@ -3,7 +3,7 @@
 <head>
   <meta charset="utf-8">
   <title>SQLUI <%= config.name %> Databases</title>
-  <link rel="icon" type="image/x-icon" href="/favicon.svg">
+  <link rel="icon" type="image/x-icon" href="<%= resource_path_map['favicon.svg'] %>">
 
   <style>
       body {
@@ -92,9 +92,9 @@
   <div class="database">
     <div class="name-and-links">
       <h2 class='name'><%= database_config.display_name %></h2>
-      <a class='query-link' href="<%= database_config.url_path %>/query">query</a>
-      <a class='saved-link' href="<%= database_config.url_path %>/saved">saved</a>
-      <a class='structure-link' href="<%= database_config.url_path %>/structure">structure</a>
+      <a class='query-link' href="<%= "#{config.base_url_path}/#{database_config.url_path}/query" %>">query</a>
+      <a class='saved-link' href="<%= "#{config.base_url_path}/#{database_config.url_path}/saved" %>">saved</a>
+      <a class='structure-link' href="<%= "#{config.base_url_path}/#{database_config.url_path}/structure" %>">structure</a>
     </div>
     <p class='description'>
       <%= database_config.description %>

data/app/views/error.erb

@@ -4,7 +4,7 @@
 <head>
   <meta charset="utf-8">
   <title><%= title %></title>
-  <link rel="icon" type="image/x-icon" href="/favicon.svg">
+  <link rel="icon" type="image/x-icon" href="<%= resource_path_map['favicon.svg'] %>">
 </head>
 <body style="font-family: monospace; font-size: 16px;">
 

data/app/views/sqlui.erb

@@ -3,7 +3,7 @@
   <head>
     <meta charset="utf-8">
     <title>SQLUI</title>
-    <link rel="icon" type="image/x-icon" href="/favicon.svg?<%= favicon_hash %>">
+    <link rel="icon" type="image/x-icon" href="<%= resource_path_map['favicon.svg'] %>">
     <!-- Initialize Airbrake before loading the main app JS so that we can catch errors as early as possible. -->
     <script type="text/javascript" src="https://cdn.jsdelivr.net/npm/@airbrake/browser"></script>
     <script type="text/javascript">
@@ -19,9 +19,9 @@
         window.airbrake?.notify(error)
       }
     </script>
-    <script type="text/javascript" src="sqlui.js?<%= js_hash %>"></script>
+    <script type="text/javascript" src="<%= resource_path_map['sqlui.js'] %>"></script>
     <script type="text/javascript" src="https://www.gstatic.com/charts/loader.js"></script>
-    <link rel="stylesheet" href="sqlui.css?<%= css_hash %>">
+    <link rel="stylesheet" href="<%= resource_path_map['sqlui.css'] %>">
   </head>
 
   <body>
@@ -43,10 +43,12 @@
       </div>
 
       <div id="submit-box" class="tab-content-element graph-element query-element" style="display: none;">
-        <input id="cancel-button" class="cancel-button" type="button" value="cancel"></input>
-        <div class="submit-fill"></div>
+        <input id="cancel-button" type="button" value="cancel"></input>
+        <div id="editor-resizer">
+          <img src="<%= resource_path_map['vertical-resizer.svg'] %>" />
+        </div>
         <div style="position: relative;">
-          <div class="submit-button-wrapper">
+          <div id="submit-button-wrapper">
             <input id="submit-button-all" class="submit-button" type="button"></input>
             <input id="submit-button-current" class="submit-button submit-button-show" type="button"></input>
             <input id="submit-dropdown-button" class="submit-dropdown-button" type="button", value="▼"></input>

data/client/resources/sqlui.css

@@ -120,27 +120,35 @@ p {
     padding: 5px;
 }
 
-.submit-fill {
+#editor-resizer {
     display: flex;
-    flex: 1
+    flex: 1;
+    cursor: row-resize;
+    flex-direction: column;
+    align-items: center;
 }
 
-.cancel-button {
+#cancel-button {
     cursor: pointer;
-    margin: 0;
     background: none;
     color: #333;
     border: 1px solid #888;
     height: 32px;
     font-size: 18px;
+    margin: 0 220px 0 0; /* To center the resizer icon. Ok, it's a hack. Get over it. */
 }
 
-.submit-button-wrapper {
+#cancel-button-spacer {
+    width: 150px;
+}
+
+#submit-button-wrapper {
     cursor: pointer;
     border: 1px solid #888;
     height: 32px;
     display: flex;
     flex-direction: row;
+    background: white;
 }
 
 .submit-button, .submit-dropdown-button {
@@ -217,7 +225,7 @@ p {
 
 .submit-dropdown-content-button:active,
 .submit-button:active,
-.cancel-button:active,
+#cancel-button:active,
 .submit-dropdown-button:active {
     background-color: #e6e6e6;
     outline: none
@@ -254,10 +262,6 @@ table tbody tr td {
   height: 21px;
 }
 
-#result-table td, #result-table th {
-    cursor: default;
-}
-
 #result-table tbody tr td abbr a {
     color: #555;
     cursor: pointer;