fix(server): trim title suggestions, fixes #162

neopostmodern · Aug 19, 2022 · a8a6402 · a8a6402
1 parent cf3e6e9
commit a8a6402
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/server/lib/methods.js b/server/lib/methods.js
@@ -65,7 +65,7 @@ export async function fetchTitleSuggestions(url) {
       const titleBag = [
         dom.window.document.querySelector('title').innerHTML,
         ...metaTitles,
-      ].map((rawTitle) => decode(rawTitle))
+      ].map((rawTitle) => decode(rawTitle).trim())
 
       return titleBag.filter(
         (title, index) => titleBag.indexOf(title) === index,

diff --git a/server/lib/resolvers.js b/server/lib/resolvers.js
@@ -151,9 +151,7 @@ const rootResolvers = {
         throw new Error(`No link with ID ${linkId}`)
       }
       try {
-        let x = await fetchTitleSuggestions(link.url)
-        console.log(link.url, x)
-        return x
+        return fetchTitleSuggestions(link.url)
       } catch (error) {
         console.warn(`Failed to fetch title suggestions for ${linkId}`, error)
         return []