From 059ff0647a100c56e9f1d9c07de9e055371ab889 Mon Sep 17 00:00:00 2001
From: Kyle Spearrin <kspearrin@users.noreply.github.com>
Date: Tue, 19 May 2020 12:16:30 -0400
Subject: [PATCH] Added option to let user control the paged param (#45)

---
 src/app/tabs/settings.component.html   | 8 ++++++++
 src/app/tabs/settings.component.ts     | 3 +++
 src/locales/en/messages.json           | 3 +++
 src/models/ldapConfiguration.ts        | 1 +
 src/services/ldap-directory.service.ts | 2 +-
 5 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/src/app/tabs/settings.component.html b/src/app/tabs/settings.component.html
index 69f96309..fb3b3edc 100644
--- a/src/app/tabs/settings.component.html
+++ b/src/app/tabs/settings.component.html
@@ -33,6 +33,14 @@ <h3 class="card-header">{{'directory' | i18n}}</h3>
                             <label class="form-check-label" for="ad">{{'ldapAd' | i18n}}</label>
                         </div>
                     </div>
+                    <div class="form-group" *ngIf="!ldap.ad">
+                        <div class="form-check">
+                            <input class="form-check-input" type="checkbox" id="pagedSearch"
+                                [(ngModel)]="ldap.pagedSearch" name="PagedSearch">
+                            <label class="form-check-label"
+                                for="pagedSearch">{{'ldapPagedResults' | i18n}}</label>
+                        </div>
+                    </div>
                     <div class="form-group">
                         <div class="form-check">
                             <input class="form-check-input" type="checkbox" id="ldapEncrypted" [(ngModel)]="ldap.ssl"
diff --git a/src/app/tabs/settings.component.ts b/src/app/tabs/settings.component.ts
index 9b46a595..a8d956a1 100644
--- a/src/app/tabs/settings.component.ts
+++ b/src/app/tabs/settings.component.ts
@@ -84,6 +84,9 @@ export class SettingsComponent implements OnInit, OnDestroy {
 
     async submit() {
         ConnectorUtils.adjustConfigForSave(this.ldap, this.sync);
+        if (this.ldap != null && this.ldap.ad) {
+            this.ldap.pagedSearch = true;
+        }
         await this.configurationService.saveOrganizationId(this.organizationId);
         await this.configurationService.saveDirectoryType(this.directory);
         await this.configurationService.saveDirectory(DirectoryType.Ldap, this.ldap);
diff --git a/src/locales/en/messages.json b/src/locales/en/messages.json
index c30c1b05..7a610585 100644
--- a/src/locales/en/messages.json
+++ b/src/locales/en/messages.json
@@ -447,6 +447,9 @@
     "ldapAd": {
         "message": "This server uses Active Directory"
     },
+    "ldapPagedResults": {
+        "message": "This server pages search results"
+    },
     "select": {
         "message": "Select"
     },
diff --git a/src/models/ldapConfiguration.ts b/src/models/ldapConfiguration.ts
index f34246b5..dd1f7556 100644
--- a/src/models/ldapConfiguration.ts
+++ b/src/models/ldapConfiguration.ts
@@ -14,4 +14,5 @@ export class LdapConfiguration {
     username: string;
     password: string;
     ad = true;
+    pagedSearch = true;
 }
diff --git a/src/services/ldap-directory.service.ts b/src/services/ldap-directory.service.ts
index 1c6ef9f2..218df6b5 100644
--- a/src/services/ldap-directory.service.ts
+++ b/src/services/ldap-directory.service.ts
@@ -293,7 +293,7 @@ export class LdapDirectoryService implements DirectoryService {
         const options: ldap.SearchOptions = {
             filter: filter,
             scope: 'sub',
-            paged: false,
+            paged: this.dirConfig.pagedSearch,
         };
         const entries: T[] = [];
         return new Promise<T[]>((resolve, reject) => {