Skip to content

fix:fix only load extensions from normal files,now can load from jar … - #140

Open
fisherman2024 wants to merge 1 commit into
alibaba:masterfrom
fisherman2024:feature/fix-extension-load
Open

fix:fix only load extensions from normal files,now can load from jar …#140
fisherman2024 wants to merge 1 commit into
alibaba:masterfrom
fisherman2024:feature/fix-extension-load

Conversation

@fisherman2024

Copy link
Copy Markdown

原先的方式只能从普通文件夹中加载extentions,当打成jar之后,将无法加载而导致运行报错。
现在修改使用PathMatchingResourcePatternResolver方式,屏蔽了jar跟普通文件夹的差异。

Close:#139

@CLAassistant

CLAassistant commented Dec 10, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@hiSandog

hiSandog commented Jul 2, 2026

Copy link
Copy Markdown

这个修复方向对 jar 内资源是对的,不过建议补一个打包后运行的回归测试。现在 classpath*:META-INF/extensions/** 会拿到所有匹配资源,最好确认不会把目录资源、空文件名或非扩展声明文件当成类名处理。测试里可以把 extension 文件放进 test jar,再验证 ExtensionManager 能从 jar 中读取实现类,同时普通文件目录加载仍然兼容。

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants