-
Notifications
You must be signed in to change notification settings - Fork 185
Fix the problem of flowcontrol plugin interceptor declaration error and failure to remove retried instances #1760
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
base: develop
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 45 files with indirect coverage changes 🚀 New features to boost your workflow:
|
action for xds not executed |
ea4b0a1
to
64ec4a1
Compare
.setApiPath(getRequestUri.apply(request)) | ||
.setHeaders(getHeaders(request)) | ||
.setMethod(getMethod.apply(request)) | ||
.setServiceName(getHeader.apply(request, ConfigConst.FLOW_REMOTE_SERVICE_NAME_HEADER_KEY)) |
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.
Don't delete this.
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.
done
Signed-off-by: hanbingleixue <[email protected]>
What type of PR is this?
Bug
What this PR does / why we need it?
Fix some bug of flowcontrol plugin
Which issue(s) this PR fixes?
Fixes #1759
Does this PR introduce a user-facing change?
No
Checklist