Merge pull request #1 from christopherhein/chore/additional-validatio…

…n-methods

Adding validation method using DNS

Makefile

@@ -1,11 +1,23 @@
-TEMPLATE=dist/aws-ecr-public.template.json
+TEMPLATE ?= dist/aws-ecr-public.template.json
+STACK_NAME ?= aws-ecr-public
+VALIDATION_DOMAIN ?= ""
+DOMAIN_NAME ?= ""
+VALIDATION_METHOD ?= "EMAIL"
+
 
 build:
 	mkdir -p dist
 	cfn-include -t -m serverless.template.yml > $(TEMPLATE)
 
 test: build
-	aws cloudformation deploy --template-file $(TEMPLATE) --stack-name aws-ecr-public --capabilities CAPABILITY_IAM
+	aws cloudformation deploy \
+		--template-file $(TEMPLATE) \
+		--stack-name $(STACK_NAME) \
+		--capabilities CAPABILITY_IAM \
+		--parameter-overrides \
+			ValidationDomain=$(VALIDATION_DOMAIN) \
+			DomainName=$(DOMAIN_NAME) \
+			ValidationMethod=$(VALIDATION_METHOD)
 
 clean:
 	rm -rf dist

README.md

@@ -30,6 +30,7 @@ If you would like to make your registries publicly available then this solution
 | -- | -- | -- |
 | DomainName | No | If provided an ACM Certificate and API Domain Name will be created
 | ValidationDomain | No | Overwrite default Validation Domain for ACM Certificate
+| ValidationMethod | Yes, *Default: EMAIL* | Allow you to use DNS instead of EMAIL for Certificate validation
 
 ## FAQ
 

serverless.template.yml

@@ -11,6 +11,11 @@ Parameters:
     Type: String
     Default: ''
     Description: If provided an ACM Certificate and API Domain Name will be created
+  ValidationMethod:
+    Type: String
+    Default: 'EMAIL'
+    AllowedValues: ["EMAIL", "DNS"]
+    Description: If provided an ACM Certificate and API Domain Name will be created
 
 Conditions:
   HasValidationDomain: !Not [ !Equals [ !Ref ValidationDomain, '' ] ]
@@ -76,6 +81,7 @@ Resources:
     Condition: HasDomainName
     Properties:
       DomainName: !Ref DomainName
+      ValidationMethod: !Ref ValidationMethod
       DomainValidationOptions:
         Fn::If:
           - HasValidationDomain