Merge pull request #410 from mboudet/master

Fix ldap sync
genouest · Oct 27, 2023 · 9f3d3ce · 9f3d3ce
2 parents f5a8429 + 994eb85
commit 9f3d3ce
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -5,9 +5,9 @@ on: ["push", "pull_request"]
 jobs:
 
   build:
-
+    if: github.repository == 'genouest/genouestaccountmanager'
     runs-on: self-hosted
- 
+
     steps:
     - uses: actions/checkout@v1
     - name: Run unit tests

diff --git a/tools/sync_from_ldap.js b/tools/sync_from_ldap.js
@@ -231,11 +231,11 @@ async function record_user(user){
         if (cur_user.lastname !=  user.sn) {
             go_user.lastname = user.sn;
         }
-        if (cur_user.uidNumber != cur_uid) {
-            go_user.uidNumber = cur_uid;
+        if (cur_user.uidnumber != cur_uid) {
+            go_user.uidnumber = cur_uid;
         }
-        if (cur_user.gidNumber != cur_gid) {
-            go_user.gidNumber = cur_gid;
+        if (cur_user.gidnumber != cur_gid) {
+            go_user.gidnumber = cur_gid;
         }
         if (cur_user.created_at != create_time) {
             go_user.created_at = create_time;