You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dolphinscheduler.apache.org by "JohnZp (via GitHub)" <gi...@apache.org> on 2024/07/17 08:18:06 UTC
[I] [Improvement] tenant supports users of other databases like ldap [dolphinscheduler]
JohnZp opened a new issue, #16338:
URL: https://github.com/apache/dolphinscheduler/issues/16338
### Search before asking
- [X] I had searched in the [issues](https://github.com/apache/dolphinscheduler/issues?q=is%3Aissue) and found no similar feature requirement.
### Description
```java
private static List<String> getUserListFromLinux() throws IOException {
List<String> userList = new ArrayList<>();
ProcessBuilder processBuilder = new ProcessBuilder("getent", "passwd");
Process process = processBuilder.start();
try (
BufferedReader bufferedReader = new BufferedReader(
new InputStreamReader(new FileInputStream("/etc/passwd")))) {
String line;
while ((line = bufferedReader.readLine()) != null) {
if (line.contains(":")) {
String[] userInfo = line.split(":");
userList.add(userInfo[0]);
}
}
}
return userList;
}
```
This method of obtaining a complete list of users is not enough. Why not use **getent passwd**?
### Are you willing to submit a PR?
- [ ] Yes I am willing to submit a PR!
### Code of Conduct
- [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Improvement] tenant supports users of other databases like ldap [dolphinscheduler]
Posted by "SbloodyS (via GitHub)" <gi...@apache.org>.
SbloodyS commented on issue #16338:
URL: https://github.com/apache/dolphinscheduler/issues/16338#issuecomment-2232804335
Would you like to fix it? @JohnZp
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Improvement] tenant supports users of other databases like ldap [dolphinscheduler]
Posted by "SbloodyS (via GitHub)" <gi...@apache.org>.
SbloodyS closed issue #16338: [Improvement] tenant supports users of other databases like ldap
URL: https://github.com/apache/dolphinscheduler/issues/16338
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Improvement] tenant supports users of other databases like ldap [dolphinscheduler]
Posted by "SbloodyS (via GitHub)" <gi...@apache.org>.
SbloodyS commented on issue #16338:
URL: https://github.com/apache/dolphinscheduler/issues/16338#issuecomment-2244086406
This command `getent ` does not exist in all environments, we can make it configurable for users to choose.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Improvement] tenant supports users of other databases like ldap [dolphinscheduler]
Posted by "JohnZp (via GitHub)" <gi...@apache.org>.
JohnZp commented on issue #16338:
URL: https://github.com/apache/dolphinscheduler/issues/16338#issuecomment-2241972656
> Can we make this to be configured? I still do not know why `getent passwd` is better than `/etc/passwd`.
In my current environment, which is MRS, user creation is managed by a unified service. The users created by this service are not present in /etc/passwd, which might make my scenario somewhat unique.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Improvement] tenant supports users of other databases like ldap [dolphinscheduler]
Posted by "JohnZp (via GitHub)" <gi...@apache.org>.
JohnZp commented on issue #16338:
URL: https://github.com/apache/dolphinscheduler/issues/16338#issuecomment-2233036418
> Would you like to fix it? @JohnZp
I will try it when I have time
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Improvement] tenant supports users of other databases like ldap [dolphinscheduler]
Posted by "ruanwenjun (via GitHub)" <gi...@apache.org>.
ruanwenjun commented on issue #16338:
URL: https://github.com/apache/dolphinscheduler/issues/16338#issuecomment-2240826671
Can we make this to be configured? I still do not know why `getent passwd` is better than `/etc/passwd`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org