Ensure list name is in lowercase

swat-sccs · Sep 4, 2023 · 1d9b504 · 1d9b504
1 parent 6b835eb
commit 1d9b504
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/functions/createMailingList.ts b/src/functions/createMailingList.ts
@@ -32,7 +32,7 @@ export const createMailingList = async (data: any) => {
 
     const [emailText, transporter] = await Promise.all([
       generateEmail('newMailingList.html', {
-        listName: data.listName,
+        listName: data.listName.toLowerCase(),
         user: data.creator,
       }),
       mailTransporter,

diff --git a/src/integration/mailman.ts b/src/integration/mailman.ts
@@ -42,7 +42,7 @@ export const getMailingList = async (name: string): Promise<any> => {
 
 export const addMailingList = async (name: string, ownerEmail: string) => {
   logger.debug(`Creating list ${name} owned by ${ownerEmail}`);
-  const createResponse = await api.post('/lists', { fqdn_listname: `${name}@sccs.swarthmore.edu` });
+  const createResponse = await api.post('/lists', { fqdn_listname: `${name.toLowerCase()}@sccs.swarthmore.edu` });
 
   if (createResponse.status != 201) {
     const errDesc =
@@ -54,7 +54,7 @@ export const addMailingList = async (name: string, ownerEmail: string) => {
   logger.debug(`Adding ${ownerEmail} as owner of mailing list ${name}`);
 
   const addOwnerResponse = await api.post('/members', {
-    list_id: `${name}.sccs.swarthmore.edu`,
+    list_id: `${name.toLowerCase()}.sccs.swarthmore.edu`,
     subscriber: ownerEmail,
     role: 'owner',
   });