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

feature: support third-party-jwt for istio access authentication #93

Merged
merged 2 commits into from
Dec 2, 2022

Conversation

xcbeyond
Copy link
Contributor

@xcbeyond xcbeyond commented Dec 2, 2022

Ⅰ. Describe what this PR did

support third-party-jwt for istio access authentication

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

本次修改主要涉及 helm安装时,相关配置的修改。通过本地环境直接helm安装(helm install higress -n higress-system higress)验证。

Ⅴ. Special notes for reviews

@CLAassistant
Copy link

CLAassistant commented Dec 2, 2022

CLA assistant check
All committers have signed the CLA.

@johnlanni
Copy link
Collaborator

jwtPolicy: "first-party-jwt"

@xcbeyond 这里也需要改一下

@xcbeyond
Copy link
Contributor Author

xcbeyond commented Dec 2, 2022

jwtPolicy: "first-party-jwt"

@xcbeyond 这里也需要改一下

@johnlanni 已修改

@codecov-commenter
Copy link

Codecov Report

Merging #93 (299a46b) into main (1e448da) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #93   +/-   ##
=======================================
  Coverage   24.80%   24.80%           
=======================================
  Files          24       24           
  Lines        4076     4076           
=======================================
  Hits         1011     1011           
  Misses       3008     3008           
  Partials       57       57           

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants