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

BaseWxMpServiceImpl.addConfigStorage方法没有设置WxMpConfigStorageHolder.set(defaultMpId); #1937

Closed
edgeowner opened this issue Dec 23, 2020 · 3 comments

Comments

@edgeowner
Copy link

edgeowner commented Dec 23, 2020

简要描述

简单概括描述下你所遇到的问题。
BaseWxMpServiceImpl中
image

getWxMpConfigStorage()方法使用WxMpConfigStorageHolder.get()时是有问题
动态添加公众号配置方法使用 this.configStorageMap.put(mpId, configStorages)是加入本地新增,并没有设置AppId ,这里会导致Thread_Local获取的数据使用不是当前使用的mpId,会导致下一个线程使用的mpId随机使用线程池中的appId

image

模块版本情况

  • WxJava 模块名: dev
  • WxJava 版本号: dev

详细描述

尽量详细描述。请不要使用截图,尽量使用文字描述,代码直接贴上来,日志则请附在后面所示区域。

@binarywang
Copy link
Member

请准确描述你的问题,如果可以直接提交PR代码即可

@edgeowner
Copy link
Author

楼主真给力

@binarywang
Copy link
Member

楼主真给力

楼主? 你是在说你自己吗?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants