服务商模式URL修改，增加服务商模式关闭订单方法 - #3063
Conversation
Sorry, something went wrong.
| @Override | ||
| public WxPayUnifiedOrderV3Result unifiedPartnerOrderV3(PartnerTradeTypeEnum tradeType, WxPayPartnerUnifiedOrderV3Request request) throws WxPayException { | ||
| if (StringUtils.isBlank(request.getSpAppid())) { | ||
| request.setSpAppid(this.getConfig().getAppId()); |
There was a problem hiding this comment.
THREAD_SAFETY_VIOLATION: Read/Write race. Non-private method BaseWxPayServiceImpl.unifiedPartnerOrderV3(...) indirectly reads without synchronization from this.configMap. Potentially races with write in method BaseWxPayServiceImpl.addConfig(...).
Reporting because another access to the same memory occurs on a background thread, although this access may not.
❗❗ 3 similar findings have been found in this PR
🔎 Expand here to view all instances of this finding| File Path | Line Number |
|---|---|
| weixin-java-pay/src/main/java/com/github/binarywang/wxpay/service/impl/BaseWxPayServiceImpl.java | 691 |
| weixin-java-pay/src/main/java/com/github/binarywang/wxpay/service/impl/BaseWxPayServiceImpl.java | 564 |
| weixin-java-pay/src/main/java/com/github/binarywang/wxpay/service/impl/BaseWxPayServiceImpl.java | 549 |
Visit the Lift Web Console to find more details in your report.
ℹ️ Expand to see all @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
| @sonatype-lift ignore | Leave out the above finding from this PR |
| @sonatype-lift ignoreall | Leave out all the existing findings from this PR |
| @sonatype-lift exclude <file|issue|path|tool> | Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Sorry, something went wrong.
|
麻烦先解决下冲突，谢谢 |
Sorry, something went wrong.
|
好的
在 2023-06-21 14:45:42，"Binary Wang" ***@***.***> 写道：
麻烦先解决下冲突，谢谢
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Sorry, something went wrong.
|
冲突已解决
在 2023-06-21 14:45:42，"Binary Wang" ***@***.***> 写道：
麻烦先解决下冲突，谢谢
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Sorry, something went wrong.
| WxPayUnifiedOrderV3Result result = this.unifiedPartnerOrderV3(tradeType, request); | ||
| //获取应用ID | ||
| String appId = StringUtils.isBlank(request.getSubAppid()) ? request.getSpAppid() : request.getSubAppid(); | ||
| return result.getPartnerPayInfo(tradeType, appId, request.getSubMchId(), this.getConfig().getPrivateKey()); |
There was a problem hiding this comment.
THREAD_SAFETY_VIOLATION: Read/Write race. Non-private method BaseWxPayServiceImpl.unifiedPartnerOrderV3(...) indirectly reads without synchronization from this.configMap. Potentially races with write in method BaseWxPayServiceImpl.addConfig(...).
Reporting because another access to the same memory occurs on a background thread, although this access may not.
ℹ️ Expand to see all @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
| @sonatype-lift ignore | Leave out the above finding from this PR |
| @sonatype-lift ignoreall | Leave out all the existing findings from this PR |
| @sonatype-lift exclude <file|issue|path|tool> | Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Sorry, something went wrong.
No description provided.