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

add LDAP and update TOC #59

Merged
merged 2 commits into from
Jul 20, 2020
Merged

add LDAP and update TOC #59

merged 2 commits into from
Jul 20, 2020

Conversation

Amber1990Zhang
Copy link
Contributor

@Amber1990Zhang Amber1990Zhang commented Jul 9, 2020

Translated from vesoft-inc/nebula-docs#111

2. 将 `auth_ldap.so` 文件放在安装路径的共享目录中。
3. 创建影子账户安装 `auth_ldap` 插件。

以 root 身份登录 **Nebula Graph**, 密码为 `nebula`,身份验证 `auth_type` 类型为 `password`:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

连接的时候不需要auth_type吧,这个是graphd的配置文件,需要在graphd启动前配置好

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个是需要的

Comment on lines 28 to 40
创建影子账户:

```ngql
nebula> CREATE USER test2 WITH PASSWORD "";
```

安装 `auth_ldap` 插件:

```ngql
nebula> INSTALL PLUGIN auth_ldap SONMAE "auth_ldap.so";
```

查看插件是否成功安装:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

创建影子账户和安装插件调换个顺序我觉得更好。安装插件只是集群初始化的时候需要,一次就好,以后就不需要了

配置完成后,您可以使用 LDAP 身份验证连接到 **Nebula Graph**,运行以下命令:

```bash
./bin/nebula -u test2 -p passwdtest2 --port 3699 --addr="127.0.0.1"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个和上边写的重复了,留一处就好,建议留这里

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't' think so

whitewum
whitewum previously approved these changes Jul 9, 2020
Copy link

@panda-sheep panda-sheep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM,Excellent!

@Amber1990Zhang Amber1990Zhang merged commit 557e924 into vesoft-inc:master Jul 20, 2020
@Amber1990Zhang Amber1990Zhang deleted the LDAP branch July 22, 2020 07:16
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.

None yet

4 participants