fix: fix inappropriate directive clean in OperateProxy function#10748
Conversation
…ter parameter matching
|
Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
感谢支持 |
…etter parameter matching (1Panel-dev#10748)
What this PR does / why we need it?
#10747
Summary of your change
由于RemoveDirective已经被大量使用,在原函数上做修改可能会有意料之外的问题。而且大部分 directive 一般都是一个 key 一个 param,并且大部分只会在一个块出现一次, 默认这样匹配效率高。
于是另起一个扩展函数RemoveDirectiveByFullParams,对于目前和以后含表达式类且可能会在一个块内出现多次的directive,使用这个函数直接检查全部参数匹配来移除。
Please indicate you've done the following: