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

Ldap 权限验证失效 #8214

Closed
wangdongyun opened this issue Apr 20, 2022 · 3 comments · Fixed by #8216
Closed

Ldap 权限验证失效 #8214

wangdongyun opened this issue Apr 20, 2022 · 3 comments · Fixed by #8216
Labels

Comments

@wangdongyun
Copy link

wangdongyun commented Apr 20, 2022

问题 描述
!!!!最近在合并官方的develop!!!!
最新的Nacos版本 2.1.0, 出现报错日志

2022-04-20 07:28:14,965 WARN Can't find auth plugin for type ldap, please add plugin to classpath or set nacos.core.auth.enabled as false

2022-04-20 07:30:02,261 WARN Can't find auth plugin for type ldap, please add plugin to classpath or set nacos.core.auth.enabled as false

2022-04-20 07:30:14,338 WARN Can't find auth plugin for type ldap, please add plugin to classpath or set nacos.core.auth.enabled as false

2022-04-20 07:30:14,988 WARN Can't find auth plugin for type ldap, please add plugin to classpath or set nacos.core.auth.enabled as false

2022-04-20 07:30:20,240 WARN Can't find auth plugin for type ldap, please add plugin to classpath or set nacos.core.auth.enabled as false

权限验证入口 AuthFilter

   Secured secured = method.getAnnotation(Secured.class);
                if (!protocolAuthService.enableAuth(secured)) {
                    chain.doFilter(request, response);
                    return;
                }

因为上面地报错日志,导致 enableAuth 一致等于 False, @secure 注解失效

其他
从SPI的加载来看,AuthPluginService 的实现类 只有 NacosAuthPluginService, 缺少比如 LdapAuthPluginService,并且放到 serviceLoader里面去,导致权限失效~~

@onewe
Copy link
Collaborator

onewe commented Apr 20, 2022

我瞅瞅

@onewe
Copy link
Collaborator

onewe commented Apr 20, 2022

@i will resolve it@

onewe added a commit to onewe/nacos that referenced this issue Apr 22, 2022
- move the config of ldap from NacosAuthConfig to LdapAuthConfig

Close alibaba#8214
onewe added a commit to onewe/nacos that referenced this issue Apr 22, 2022
onewe added a commit to onewe/nacos that referenced this issue Apr 22, 2022
@yangzhongj
Copy link

bug 真多啊

KomachiSion pushed a commit that referenced this issue Apr 25, 2022
* [ISSUE #8214] Add ldap auth plugin

- move the config of ldap from NacosAuthConfig to LdapAuthConfig

Close #8214

* [ISSUE #8214] Resolve CI error

* [ISSUE #8214] The constants of Ldap move to the plugin-impl module
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants