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

Automated cherry pick of #5818: feat: hepa support mse para-sign-auth plugin #5819

Conversation

erda-bot
Copy link
Member

Cherry pick of #5818 on release/2.3.

Squashed commit message:

commit e15d0b16d2ab73cbc0b29708777940024e909b2d
Author: wang zhuzhen <zhuzhenwang1984@gmail.com>
Date:   Tue Apr 18 09:59:33 2023 +0800

    feat: hepa support mse para-sign-auth plugin (#5818)

What this PR does / why we need it:

Hepa supports MSE custom plugin para-sign-auth

Specified Reviewers:

/assign @dspo @iutx @sixther-dc @luobily

ChangeLog

Feature: Support MSE custom plugin para-sign-auth in hepa(Erda 网关支持 MSE 自定义鉴权插件 para-sign-auth)

Language Changelog
🇺🇸 English Support MSE custom plugin para-sign-auth in hepa
🇨🇳 中文 Erda 网关支持 MSE 自定义鉴权插件 para-sign-auth

Need cherry-pick to release versions?

/cherry-pick release/2.3

@erda-bot
Copy link
Member Author

Automated cherry pick can ONLY be triggered when this PR is MERGED!

@codecov
Copy link

codecov bot commented Apr 18, 2023

Codecov Report

Merging #5819 (e15d0b1) into release/2.3 (51d2570) will increase coverage by 0.01%.
The diff coverage is 30.76%.

Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff               @@
##           release/2.3    #5819      +/-   ##
===============================================
+ Coverage        22.17%   22.19%   +0.01%     
===============================================
  Files             1837     1838       +1     
  Lines           203747   203870     +123     
===============================================
+ Hits             45187    45242      +55     
- Misses          153366   153419      +53     
- Partials          5194     5209      +15     
Impacted Files Coverage Δ
...rchestrator/hepa/apipolicy/policies/sbac/policy.go 9.34% <0.00%> (-0.09%) ⬇️
.../hepa/gateway-providers/dto/credential_resp_dto.go 3.77% <ø> (ø)
...estrator/hepa/gateway-providers/mse/mse_openapi.go 1.48% <0.00%> (-0.04%) ⬇️
...hepa/gateway-providers/mse/plugins/mse_key_auth.go 84.50% <ø> (+30.45%) ⬆️
...chestrator/hepa/services/endpoint_api/impl/impl.go 2.32% <0.00%> (-0.04%) ⬇️
...rator/hepa/gateway-providers/mse/plugins/common.go 39.14% <9.43%> (-8.01%) ⬇️
...estrator/hepa/gateway-providers/mse/mse_adapter.go 61.55% <31.81%> (-3.07%) ⬇️
...trator/hepa/services/openapi_consumer/impl/impl.go 9.05% <46.15%> (-0.21%) ⬇️
...ateway-providers/mse/plugins/mse_para_sign_auth.go 68.85% <68.85%> (ø)
...epa/gateway-providers/mse/plugins/mse_hmac_auth.go 87.20% <90.00%> (ø)

... and 3 files with indirect coverage changes

@sixther-dc sixther-dc merged commit 4d067b6 into erda-project:release/2.3 Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants