Merge pull request #936 from live-codes/fix-meta-tags

fix meta tags

Author: hatemhosny

functions/index.ts

@@ -1,4 +1,4 @@
-import { getProjectInfo, type Context, type PgFunction } from './utils.ts';
+import { encodeHTML, getProjectInfo, type Context, type PgFunction } from './utils.ts';
 
 export const onRequest: PgFunction = async function (context) {
   const { request, env } = context;
@@ -48,15 +48,15 @@ export const onRequest: PgFunction = async function (context) {
       .replace(
         /title" content="LiveCodes"/g,
         `title" content="${
-          !title || title === 'Untitled Project' ? 'LiveCodes' : title + ' - LiveCodes'
+          !title || title === 'Untitled Project' ? 'LiveCodes' : encodeHTML(title) + ' - LiveCodes'
         }"`,
       )
       .replace(
         /content="A Code Playground That Just Works!"/g,
         `content="${
           !title && !description
             ? 'A Code Playground That Just Works!'
-            : description || 'A project on LiveCodes.'
+            : encodeHTML(description || 'A project on LiveCodes.')
         }"`,
       )
       .replace(/content="https:\/\/livecodes.io\/"/g, `content="${request.url}"`)
@@ -77,7 +77,7 @@ export const onRequest: PgFunction = async function (context) {
 
     context.waitUntil(logToAPI(context));
     return response;
-  } catch (err) {
+  } catch (err: any) {
     context.data = {
       ...data,
       ok: false,

functions/utils.ts

@@ -80,3 +80,11 @@ export const getProjectInfo = async (url: URL): Promise<ProjectInfo> => {
     description: '',
   };
 };
+
+export const encodeHTML = (html: string) =>
+  html
+    .replace(/&/g, '&amp;')
+    .replace(/</g, '&lt;')
+    .replace(/>/g, '&gt;')
+    .replace(/'/g, '&#39;')
+    .replace(/"/g, '&#34;');