Territory links are case-sensitive

[ekzyis]

Description

Territories links are case-sensitive. This means that https://stacker.news/~Psychedelics works but https://stacker.news/~psychedelics does not.

Steps to Reproduce

Click on https://stacker.news/~psychedelics -> 404
Click on https://stacker.news/~Psychedelics -> 200

Expected behavior

Both links should work.

Additional context

Since we use citext, we already made sure that all queries hitting Sub.name are implicitly case-insensitive. This means we already made sure that territories with different cases cannot be created. But for some reason, the links are still case-sensitive.

https://stacker.news/items/361271

[ekzyis]

For some reason, the sub resolver seems to not be case-insensitive. With the following patch, https://stacker.news/~psychedelics finds the sub but other queries seem to fail (since the dropdown does not reflect the sub we're in)

diff --git a/api/resolvers/sub.js b/api/resolvers/sub.js
index 5d8923d..65063bd 100644
--- a/api/resolvers/sub.js
+++ b/api/resolvers/sub.js
@@ -76,9 +76,12 @@ export default {
         }).catch(console.log)
       }

-      return await models.sub.findUnique({
+      return await models.sub.findFirst({
         where: {
-          name
+          name: {
+            equals: name,
+            mode: 'insensitive'
+          }
         }
       })
     },

video

2023-12-22.11-44-02.mp4

But Prisma seems to be aware of citext ...