Remove try/catch for short url so the appropriate errors will be propagated to the UI

src/server/http/short_url_error.js

@@ -1,9 +1,8 @@
 import Boom from 'boom';
 
-export function handleShortUrlError(err) {
-  if (err.isBoom) return err;
-  if (err.status === 401) return Boom.unauthorized();
-  if (err.status === 403) return Boom.forbidden();
-  if (err.status === 404) return Boom.notFound();
-  return Boom.badImplementation();
+export function handleShortUrlError(error) {
+  if (!error.isBoom && !(error instanceof Error)) {
+    return Boom.create(error.status || 500);
+  }
+  return Boom.wrap(error, error.status || 500);
 }

src/server/http/short_url_lookup.js

@@ -37,14 +37,10 @@ export default function (server) {
     },
 
     async getUrl(id, req) {
-      try {
-        const doc = await req.getSavedObjectsClient().get('url', id);
-        updateMetadata(doc, req);
+      const doc = await req.getSavedObjectsClient().get('url', id);
+      updateMetadata(doc, req);
 
-        return doc.attributes.url;
-      } catch (err) {
-        return '/';
-      }
+      return doc.attributes.url;
     }
   };
 }