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

feat(ai-proxy): add client token; add rate-limit filter #6045

Merged
merged 8 commits into from
Sep 19, 2023

Conversation

sfwn
Copy link
Member

@sfwn sfwn commented Sep 19, 2023

What this PR does / why we need it:

  • add client-token
  • remove useless configs
  • add rate-limit filter

Specified Reviewers:

/assign @your-reviewer

ChangeLog

Language Changelog
🇺🇸 English add client token
🇨🇳 中文 增加 client token

@sfwn sfwn changed the title feat(ai-proxy): add client token feat(ai-proxy): add client token; add rate-limit filter Sep 19, 2023
@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

Merging #6045 (56b56ed) into master (852a5e3) will increase coverage by 0.30%.
Report is 11 commits behind head on master.
The diff coverage is 46.07%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6045      +/-   ##
==========================================
+ Coverage   22.48%   22.79%   +0.30%     
==========================================
  Files        1888     1767     -121     
  Lines      209748   190738   -19010     
==========================================
- Hits        47170    43477    -3693     
+ Misses     157232   142272   -14960     
+ Partials     5346     4989     -357     
Files Changed Coverage Δ
...ocol/standard-components/issueFilter/conditions.go 6.71% <0.00%> (-0.19%) ⬇️
...otocol/standard-components/issueFilter/provider.go 13.33% <0.00%> (-0.28%) ⬇️
...viders/issue/core/query/issueexcel/model_sample.go 0.00% <0.00%> (ø)
...nal/apps/dop/providers/project_report/calculate.go 6.34% <0.00%> (+0.02%) ⬆️
...nternal/apps/dop/providers/project_report/query.go 17.60% <0.00%> (-0.44%) ⬇️
...al/pkg/ai-functions/handler/handler_ai_function.go 0.00% <0.00%> (ø)
internal/pkg/ai-functions/sdk/client.go 0.00% <0.00%> (ø)
pkg/http/httputil/header.go 94.44% <ø> (ø)
...l/pkg/ai-functions/functions/test-case/function.go 45.58% <58.00%> (ø)
...ons/handler/handler_ai_function_create_testcase.go 64.04% <64.04%> (ø)
... and 1 more

... and 128 files with indirect coverage changes

@sfwn sfwn merged commit 3b2bb5c into erda-project:master Sep 19, 2023
27 checks passed
jing-zhi pushed a commit to jing-zhi/erda that referenced this pull request Sep 25, 2023
…#6045)

* add api for client-token

* support check client token

* polish token create: add metadata; support createOrGet

* remove useless configs: providers/platforms

* fulfill audit from client token

* add rate-limit filter for client token

* fix golangci-lint

* comment out CODE-TEST cmp temporarily
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants