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.

Sign up for GitHub

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

Jump to bottom

[YSQL] LDAP Integration for YSQL #6088

Closed
m-iancu opened this issue Oct 18, 2020 · 0 comments
Closed

[YSQL] LDAP Integration for YSQL #6088

m-iancu opened this issue Oct 18, 2020 · 0 comments
Assignees
@d-uspenskiy
Labels
area/ysql Yugabyte SQL (YSQL)
Milestone
v2.3

Comments

@m-iancu
Copy link
Contributor

m-iancu commented Oct 18, 2020

This would be similar to (and based on) the PostgreSQL LDAP authentication support.
@m-iancu m-iancu added this to the v2.3 milestone Oct 18, 2020
@m-iancu m-iancu self-assigned this Oct 18, 2020
@m-iancu m-iancu mentioned this issue Oct 18, 2020
Closed
@m-iancu m-iancu assigned d-uspenskiy and unassigned m-iancu Oct 28, 2020
@m-iancu m-iancu added the area/ysql Yugabyte SQL (YSQL) label Oct 28, 2020
d-uspenskiy added a commit that referenced this issue Oct 29, 2020
@d-uspenskiy
[YSQL] #5459 #6088 #2415 Include ldap and uuid libs in default builds
c1462a1 
Summary:
Include support in all default builds for:

- LDAP Authentication (by including ldap lib in the build)
- UUID (by including uuid-ossp extension with e2fs implementation)

Additional change:
Add `ysql_pg_conf_csv`/`ysql_hba_conf_csv` flags as substitutions for deprecated `ysql_pg_conf` and `ysql_hba_conf`.
Example how to start the cluster with LDAP authentication

```
./bin/yb-ctl start --ysql_hba_conf_csv='host all yugabyte 127.0.0.1/0 password,"host   all         all      0.0.0.0/0  ldap ldapserver=ldap.forumsys.com ldapprefix=""uid="" ldapsuffix="", dc=example, dc=com"" ldapport=389"'
```

Test Plan:
Jenkins:
Existing tests, additional tests in:

TestPgRegressExtension (yb_extensions),

Reviewers: sanketh, mihnea

Reviewed By: mihnea

Subscribers: mikhail, rskannan, yql

Differential Revision: https://phabricator.dev.yugabyte.com/D9692
m-iancu pushed a commit that referenced this issue Nov 1, 2020
@d-uspenskiy @m-iancu
[Backport to 2.3][YSQL] #5459 #6088 #2415 Include ldap and uuid libs …
e8ad05e 
…in default builds

Summary:
Include support in all default builds for:

- LDAP Authentication (by including ldap lib in the build)
- UUID (by including uuid-ossp extension with e2fs implementation)

Additional change:
Add `ysql_pg_conf_csv`/`ysql_hba_conf_csv` flags as substitutions for deprecated `ysql_pg_conf` and `ysql_hba_conf`.
Example how to start the cluster with LDAP authentication

```
./bin/yb-ctl start --ysql_hba_conf_csv='host all yugabyte 127.0.0.1/0 password,"host   all         all      0.0.0.0/0  ldap ldapserver=ldap.forumsys.com ldapprefix=""uid="" ldapsuffix="", dc=example, dc=com"" ldapport=389"'
```

Test Plan:
Jenkins: rebase: 2.3
Existing tests, additional tests in:

TestPgRegressExtension (yb_extensions),

Reviewers: sanketh, dmitry, rskannan

Reviewed By: rskannan

Subscribers: yql, rskannan, mikhail

Differential Revision: https://phabricator.dev.yugabyte.com/D9787
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@d-uspenskiy d-uspenskiy

Labels
area/ysql Yugabyte SQL (YSQL)
Projects
None yet
Milestone
v2.3
Development

No branches or pull requests
2 participants
@m-iancu @d-uspenskiy