Skip to content

Commit

Permalink
fix(ldap-check): use user_name_attribute parameter
Browse files Browse the repository at this point in the history
While user_name_attribute was properly used by slurm-web-gateway, it
wasn't actually used by slurm-web-ldap-check utility. This is fixed by
this commit.

fix #340
  • Loading branch information
rezib committed Aug 26, 2024
1 parent 9efea58 commit 4f1d5df
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Detect responses from slurmrestd not formatted in JSON, translated into JSON
error for frontend and emit clear error message in logs (#333).
- genjwt: fix portability to Python < 3.8 in debug message.
- ldap-check: fix usage of `user_name_attribute` configuration parameter (#340).
- frontend:
- Support node names without digits in expand/fold logic (#328).
- Update dependencies to fix CVE-2024-39338 (axios) and CVE-2024-6783
Expand Down
1 change: 1 addition & 0 deletions slurmweb/apps/ldap.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ def run(self):
user_base=self.settings.ldap.user_base,
user_class=self.settings.ldap.user_class,
group_base=self.settings.ldap.group_base,
user_name_attribute=self.settings.ldap.user_name_attribute,
user_fullname_attribute=self.settings.ldap.user_fullname_attribute,
group_name_attribute=self.settings.ldap.group_name_attribute,
starttls=self.settings.ldap.starttls,
Expand Down

0 comments on commit 4f1d5df

Please sign in to comment.