Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added support for Cloud front origin request policy #656

Merged
merged 20 commits into from
Oct 28, 2021

Conversation

Nirbhay1997
Copy link
Contributor

@Nirbhay1997 Nirbhay1997 commented Sep 30, 2021

Description

It contains the singular resource of the above resource. It does not have a plural resource.

Issues Resolved

This is a new resource. There are no issues to be fixed.

Check List

Please fill box or appropriate ([x]) or mark N/A.

Signed-off-by: Nirbhay Kumar <nkumar@progress.com>
@Nirbhay1997 Nirbhay1997 requested review from a team as code owners September 30, 2021 10:36
Nirbhay Kumar and others added 9 commits September 30, 2021 16:52
Signed-off-by: Nirbhay Kumar <nkumar@progress.com>
Signed-off-by: Soumyodeep Karmakar <soumyo.k13@gmail.com>
Signed-off-by: Soumyodeep Karmakar <soumyo.k13@gmail.com>
Signed-off-by: Soumyodeep Karmakar <soumyo.k13@gmail.com>
Signed-off-by: Soumyodeep Karmakar <soumyo.k13@gmail.com>
Signed-off-by: Soumyodeep Karmakar <soumyo.k13@gmail.com>
Signed-off-by: Soumyodeep Karmakar <soumyo.k13@gmail.com>
Signed-off-by: Soumyodeep Karmakar <soumyo.k13@gmail.com>
Nirbhay Kumar and others added 3 commits September 30, 2021 21:33
Signed-off-by: Nirbhay Kumar <nkumar@progress.com>
Signed-off-by: Nirbhay Kumar <nkumar@progress.com>
Copy link
Contributor

@soumyo13 soumyo13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sonarcloud
Copy link

sonarcloud bot commented Sep 30, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Nirbhay Kumar and others added 4 commits September 30, 2021 21:36
Signed-off-by: Nirbhay Kumar <nkumar@progress.com>
Signed-off-by: Soumyodeep Karmakar <soumyo.k13@gmail.com>
@IanMadd
Copy link
Contributor

IanMadd commented Oct 7, 2021

CloudFront is one word so it shouldn't have an underscore between "cloud" and "front" in the resource name.

IanMadd and others added 2 commits October 7, 2021 15:29
Signed-off-by: Ian Maddaus <Ian.Maddaus@progress.com>
Signed-off-by: Nirbhay Kumar <nkumar@progress.com>
@Nirbhay1997
Copy link
Contributor Author

CloudFront is one word so it shouldn't have an underscore between "cloud" and "front" in the resource name.

Done with the changes , Please verify.

@Nirbhay1997
Copy link
Contributor Author

@IanMadd Can you please verify ??

Signed-off-by: Soumyodeep Karmakar <soumyo.k13@gmail.com>
@sonarcloud
Copy link

sonarcloud bot commented Oct 28, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@soumyo13 soumyo13 merged commit 4ab270c into main Oct 28, 2021
@soumyo13 soumyo13 deleted the Nirbhay/OriginRequestPolicy branch October 28, 2021 05:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AWS::CloudFront::OriginRequestPolicy
3 participants