Allow for AD range attribute option

ldapjs · Nov 7, 2023 · 0faadb6 · 0faadb6
1 parent cdd90b4
commit 0faadb6
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/lib/messages/search-request.js b/lib/messages/search-request.js
@@ -35,7 +35,7 @@ const isValidAttributeString = str => {
     return true
   }
   // ascii attribute names
-  if (/^[a-zA-Z][\w\d.;-]*$/.test(str) === true) {
+  if (/^[a-zA-Z][\w\d.;=*-]*$/.test(str) === true) {
     return true
   }
   return false

diff --git a/lib/messages/search-request.test.js b/lib/messages/search-request.test.js
@@ -111,6 +111,13 @@ tap.test('.attributes', t => {
     t.strictSame(req.attributes, ['a'])
   })
 
+  t.test('supports range options', async t => {
+    const req = new SearchRequest({
+      attributes: ['a;range=0-*']
+    })
+    t.strictSame(req.attributes, ['a;range=0-*'])
+  })
+
   t.test('skip empty attribute name', async t => {
     process.on('warning', handler)
     t.teardown(async () => {