-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
HTTP API: CORS configuration #7336
Conversation
Codecov Report
@@ Coverage Diff @@
## master #7336 +/- ##
==========================================
+ Coverage 87.92% 87.96% +0.03%
==========================================
Files 240 240
Lines 8898 8925 +27
==========================================
+ Hits 7824 7851 +27
Misses 1074 1074
Continue to review full report at Codecov.
|
a33d75b
to
92f4511
Compare
92f4511
to
07ac90a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Very clear implementation 👌
LGTM
@@ -46,9 +46,13 @@ describe('HttpApiEvents', () => { | |||
|
|||
it('Should not configure default route', () => { | |||
const resource = cfResources[naming.getHttpApiLogicalId()]; | |||
expect(resource.Properties).to.not.have.property('RputeKey'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
53d1027
to
3083970
Compare
3083970
to
5818494
Compare
Addresses another implementation step as outlined at: #7052