Merge pull request #719 from tadast/tt/404

yannickwurm · web-flow · commit a1d4dca043f2 · 2024-01-24T10:15:07.000Z
Add a dedicated 404 page
diff --git a/lib/sequenceserver/routes.rb b/lib/sequenceserver/routes.rb
@@ -137,7 +137,7 @@ class Routes < Sinatra::Base
     # the results.
     get '/:jid' do |jid|
       job = Job.fetch(jid)
-      raise NotFound, 'Job not found' if job.nil?
+      halt 404, File.read(File.join(settings.root, 'public/404.html')) if job.nil?
 
       erb :report, layout: true
     end
diff --git a/public/404.html b/public/404.html
@@ -0,0 +1,27 @@
+<!DOCTYPE html>
+<html lang="en">
+
+<head>
+  <meta charset="utf-8">
+  <meta name="viewport" content="width=device-width, initial-scale=1">
+
+  <title>SequenceServer: Not Found</title>
+
+  <!-- Bootstrap CSS -->
+  <link href="https://cdn.jsdelivr.net/npm/bootstrap@5.0.2/dist/css/bootstrap.min.css" rel="stylesheet"
+    integrity="sha384-EVSTQN3/azprG1Anm3QDgpJLIm9Nao0Yz1ztcQTwFspd3yD65VohhpuuCOmLASjC" crossorigin="anonymous">
+</head>
+
+<body class="bg-light text-dark text-center">
+
+  <img src="SequenceServer_logo.png" alt="SequenceServer Logo" class="mt-5 mb-3" style="max-width: 200px;">
+
+  <div class="container" style="margin-top: 10%;">
+    <h1 class="display-4">404 Not Found</h1>
+    <p class="lead">Sorry, the page you are looking for could not be found.</p>
+    <a href="/" class="btn btn-primary mt-3">Start over</a>
+  </div>
+
+</body>
+
+</html>
