From 4cdf54d0495c6f5ced4639ae5382e759d9596beb Mon Sep 17 00:00:00 2001
From: Harrison Rose <sevenlayercookie@gmail.com>
Date: Mon, 22 Jul 2024 23:14:50 -0500
Subject: [PATCH] doesn't need to be a public route (should go behind usual API
 auth)

---
 server/controllers/ShareController.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/server/controllers/ShareController.js b/server/controllers/ShareController.js
index 36171df49..095a84666 100644
--- a/server/controllers/ShareController.js
+++ b/server/controllers/ShareController.js
@@ -238,10 +238,9 @@ class ShareController {
     res.sendStatus(204)
   }
   /**
-   * Public route
    * GET: /api/share/mediaitem/:id
    * get existing share from mediaItemId
-   *  (or would it be better to return just the slug?)
+   *  (or would it be better to return just the slug instead of whole share object?)
    * 
    * @param {import('express').Request} req
    * @param {import('express').Response} res
@@ -261,6 +260,7 @@ class ShareController {
 
       if (existingMediaItemShare) {
         return res.status(200).json(existingMediaItemShare.toJSONForClient()); // or would it be better to return just the slug?
+      //return res.status(200).json(existingMediaItemShare.slug.toJSONForClient());
       } else {
         return res.status(404).send('Share not found');
       }