feat: added exception for duplicate domain

packages/deployment-service/src/dns/dns.controller.test.ts

@@ -41,6 +41,7 @@ const mockPipelineProvider = {
     }),
   ),
   update: jest.fn((pipeline) => Promise.resolve(pipeline)),
+  findByDomain: jest.fn(() => Promise.resolve(null)),
 }
 
 const mockDnsProvider = {

packages/deployment-service/src/dns/dns.controller.ts

@@ -40,11 +40,15 @@ export class DnsController {
 
     this.ownershipProvider.check(pipeline, creds.developerId as string)
 
+    const domainExists = await this.pipelineProvider.findByDomain(body.customDomain)
+
+    if (domainExists) throw new UnprocessableEntityException()
+
     const verifyDnsName = 'reapit-iaas'
     const verifyDnsValue = uuid()
 
     await this.pipelineProvider.update(pipeline, {
-      customDomain: body.customDomain, // TODO should strip everything not a domain? query params example
+      customDomain: body.customDomain,
       verifyDnsValue,
       verifyDnsName,
     })

packages/deployment-service/src/pipeline/pipeline-provider.ts

@@ -138,4 +138,10 @@ export class PipelineProvider {
       relations: ['repository'],
     })
   }
+
+  async findByDomain(customDomain: string): Promise<PipelineEntity | null> {
+    return this.repository.findOneBy({
+      customDomain,
+    })
+  }
 }