-
Notifications
You must be signed in to change notification settings - Fork 435
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
Add e2e test for cors annotations #171 #179
Conversation
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.
Thanks for picking this up @Hinsteny. Please refer to current higress supported cors annotations to cover them all.
nginx.ingress.kubernetes.io/enable-cors | Ingress | 兼容 | 开启或关闭跨域 |
---|---|---|---|
nginx.ingress.kubernetes.io/cors-allow-origin | Ingress | 兼容 | 允许的第三方站点 |
nginx.ingress.kubernetes.io/cors-allow-methods | Ingress | 兼容 | 允许的请求方法,如GET、POST、PUT等 |
nginx.ingress.kubernetes.io/cors-allow-headers | Ingress | 兼容 | 允许的请求Header |
nginx.ingress.kubernetes.io/cors-expose-headers | Ingress | 兼容 | 允许的暴露给浏览器的响应Header |
nginx.ingress.kubernetes.io/cors-allow-credentials | Ingress | 兼容 | 是否允许携带凭证信息 |
nginx.ingress.kubernetes.io/cors-max-age | Ingress | 兼容 | 预检结果的最大缓存时间 |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #179 +/- ##
==========================================
- Coverage 42.41% 42.39% -0.03%
==========================================
Files 29 29
Lines 4866 4866
==========================================
- Hits 2064 2063 -1
- Misses 2641 2642 +1
Partials 161 161
|
65136ee
to
fcf089c
Compare
Thanks @Hinsteny, ping me when you are ready for review. |
fcf089c
to
bb20386
Compare
@Xunzhuo Please help to Review it again |
bb20386
to
349eec7
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.
Generally LGTM, thank you @Hinsteny !
Ⅰ. Describe what this PR did
Add e2e test for cors annotations #171
Ⅱ. Does this pull request fix one issue?
NO
Ⅲ. Why don't you add test cases (unit test/integration test)?
this is a test PR
Ⅳ. Describe how to verify it
verify by make e2e test
Ⅴ. Special notes for reviews