-
Notifications
You must be signed in to change notification settings - Fork 35
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(controller): add domain names for object storage #3029
Conversation
01d8a6d
to
09d7246
Compare
09d7246
to
c3fc2c7
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #3029 +/- ##
============================================
- Coverage 82.94% 73.53% -9.41%
- Complexity 3168 3205 +37
============================================
Files 568 433 -135
Lines 31490 15638 -15852
Branches 1850 1809 -41
============================================
- Hits 26119 11500 -14619
+ Misses 4552 3316 -1236
- Partials 819 822 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
...rc/main/java/ai/starwhale/mlops/configuration/security/ObjectStoreDomainDetectionFilter.java
Show resolved
Hide resolved
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.
LGTM
Description
recognize regex pattern for the object store domain in the header
client part:
#3048
Test cases
Modules
Checklist