pmcf 6.28.3 → 6.29.0

package/README.md

@@ -57,12 +57,12 @@ generates config packages for:
         *   [Parameters](#parameters-6)
     *   [property](#property)
         *   [Parameters](#parameters-7)
+    *   [enabled](#enabled)
     *   [priority](#priority)
     *   [expression](#expression)
         *   [Parameters](#parameters-8)
     *   [templateContent](#templatecontent)
         *   [Parameters](#parameters-9)
-    *   [enabled](#enabled)
     *   [isTemplate](#istemplate)
     *   [expand](#expand)
         *   [Parameters](#parameters-10)
@@ -78,30 +78,30 @@ generates config packages for:
 *   [domainNames](#domainnames)
 *   [InitializationContext](#initializationcontext)
     *   [Parameters](#parameters-14)
+*   [SkeletonNetworkInterface](#skeletonnetworkinterface)
+    *   [networkAddresses](#networkaddresses)
+        *   [Parameters](#parameters-15)
 *   [zones](#zones)
 *   [type](#type)
 *   [addressesStatement](#addressesstatement)
-    *   [Parameters](#parameters-15)
+    *   [Parameters](#parameters-16)
 *   [SystemdJournalRemoteService](#systemdjournalremoteservice)
     *   [Properties](#properties)
     *   [systemdConfigs](#systemdconfigs)
-        *   [Parameters](#parameters-16)
+        *   [Parameters](#parameters-17)
 *   [SystemdJournalUploadService](#systemdjournaluploadservice)
     *   [Properties](#properties-1)
     *   [systemdConfigs](#systemdconfigs-1)
-        *   [Parameters](#parameters-17)
+        *   [Parameters](#parameters-18)
 *   [NetworkAddress](#networkaddress)
-    *   [Parameters](#parameters-18)
+    *   [Parameters](#parameters-19)
     *   [subnet](#subnet)
     *   [networkInterface](#networkinterface)
     *   [address](#address)
 *   [addresses](#addresses)
-    *   [Parameters](#parameters-19)
-*   [cidrAddresses](#cidraddresses)
     *   [Parameters](#parameters-20)
-*   [SkeletonNetworkInterface](#skeletonnetworkinterface)
-    *   [networkAddresses](#networkaddresses)
-        *   [Parameters](#parameters-21)
+*   [cidrAddresses](#cidraddresses)
+    *   [Parameters](#parameters-21)
 *   [families](#families)
 *   [secretName](#secretname)
 *   [directHosts](#directhosts)
@@ -196,6 +196,10 @@ Returns **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/G
 
 Returns **any** 
 
+### enabled
+
+Returns **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)** 
+
 ### priority
 
 Returns **[number](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Number)** 
@@ -218,10 +222,6 @@ Returns **any** 
 
 Returns **AsyncIterable\<ContentProvider>**&#x20;
 
-### enabled
-
-Returns **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)**&#x20;
-
 ### isTemplate
 
 Returns **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)**&#x20;
@@ -296,6 +296,18 @@ Keeps track of all in flight object creations and loose ends during config initi
 
 *   `directory`   (optional, default `"/"`)
 
+## SkeletonNetworkInterface
+
+**Extends ServiceOwner**
+
+### networkAddresses
+
+#### Parameters
+
+*   `filter` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)**  (optional, default `n=>true`)
+
+Returns **Iterable<[NetworkAddress](#networkaddress)>**&#x20;
+
 ## zones
 
 Type: [Array](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Array)\<bind\_zone>
@@ -392,18 +404,6 @@ Returns **Iterable<[string](https://developer.mozilla.org/docs/Web/JavaScript/Re
 
 Returns **[Array](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Array)<[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)>**&#x20;
 
-## SkeletonNetworkInterface
-
-**Extends ServiceOwner**
-
-### networkAddresses
-
-#### Parameters
-
-*   `filter` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)**  (optional, default `n=>true`)
-
-Returns **Iterable<[NetworkAddress](#networkaddress)>**&#x20;
-
 ## families
 
 Returns **[Set](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Set)<[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)>**&#x20;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pmcf",
-  "version": "6.28.3",
+  "version": "6.29.0",
   "publishConfig": {
     "access": "public"
   },
@@ -53,7 +53,7 @@
     "aggregated-map": "^1.0.10",
     "content-entry-transform": "^1.7.0",
     "ip-utilties": "^3.12.0",
-    "npm-pkgbuild": "^20.8.14",
+    "npm-pkgbuild": "^20.8.15",
     "pacc": "^11.2.2",
     "package-directory": "^8.2.0",
     "yaml": "^2.9.0"

package/src/base.mjs

@@ -166,7 +166,7 @@ export class Base {
       if (typeof value?.get === "function") {
         const object = value.get(first);
         if (object) {
-          return parts.length >= 1 ? object.named(parts.join("/")) : object;
+          return parts.length === 0 ? object : object.named(parts.join("/"));
         }
       }
     }

package/src/initialization-context.mjs

@@ -33,6 +33,12 @@ export class InitializationContext {
         }
       }
 
+      const resolved = object.expression(value);
+      if (resolved) {
+        assign(attribute, object, resolved);
+        continue nextOutstanding;
+      }
+
       this.error(
         `Unknown ${attribute.name}(${attribute.type.name}): "${value}"`
       );
@@ -178,7 +184,11 @@ export class InitializationContext {
       attribute => attribute.type === type && attribute.collection
     )) {
       try {
-        return assign(attribute, owner, this.read(create(type, owner, data), data));
+        return assign(
+          attribute,
+          owner,
+          this.read(create(type, owner, data), data)
+        );
       } catch (e) {
         this.error(`Unable to assign ${type.name}.${attribute.name}`, e, data);
       }

package/src/network-address.mjs

@@ -100,6 +100,13 @@ export class NetworkAddress {
  * @returns {Iterable<string>} addresses
  */
 export function addresses(sources, options) {
+  if (sources === undefined) {
+    return [];
+  }
+  if(sources instanceof Owner) {
+    sources = [sources];
+  }
+
   return [
     ...new Set(
       [...sources]

package/src/services/bind.mjs

@@ -15,8 +15,7 @@ import {
   boolean_attribute_writable_true,
   boolean_attribute_writable_false,
   integer_attribute_writable,
-  integer_attribute,
-  name_attribute_writable
+  integer_attribute
 } from "pacc";
 import {
   Base,
@@ -158,12 +157,7 @@ class bind_zone_config extends Base {
         content.push(`  type ${this.type};`);
         content.push(`  file \"${zone.file}\";`);
         content.push(
-          addressesStatement(
-            "allow-update",
-            group.allowedUpdates,
-            "none;",
-            "  "
-          )
+          addressesStatement("allow-update", group.allowUpdate, "none;", "  ")
         );
         content.push(`  notify ${yesno(group.notify)};`);
       }
@@ -189,23 +183,91 @@ class bind_zone_config extends Base {
   }
 }
 
-class bind_group extends Base {
+class bind_object extends Base {
   static priority = 1;
   static attributes = {
-    name: name_attribute_writable,
     order: { ...integer_attribute, name: "order" },
-    access: {
-      type: bindNetworkAddressTypes,
-      name: "access",
-      collection: true,
-      writable: true,
-      deferredExpression: true
-    },
     entries: {
       ...default_collection_attribute_writable,
       type: NetworkAddress,
       name: "entries",
       deferredExpression: true
+    }
+  };
+
+  static {
+    addType(this);
+  }
+
+  get order() {
+    return 0;
+  }
+
+  get service() {
+    return this.owner;
+  }
+}
+
+export class bind_acl extends bind_object {
+  static priority = 1;
+
+  static {
+    addType(this);
+  }
+
+  async packageContent(outputControl) {
+    const acls = addressesStatement(
+      `acl ${this.name}`,
+      addresses(this.entries, { aggregate: true })
+    );
+
+    if (acls.length) {
+      await writeLines(
+        join(outputControl.dir, "etc/named"),
+        `${this.order}-acl-${this.name}.conf`,
+        acls
+      );
+
+      return true;
+    }
+
+    return false;
+  }
+}
+
+const acl_attribute = {
+  ...default_attribute_writable,
+  type: bind_acl,
+  name: "acl",
+  default: "'any'"
+};
+
+class bind_group extends bind_object {
+  static priority = 1;
+  static attributes = {
+    matchClients: {
+      ...acl_attribute,
+      name: "matchClients"
+    },
+    allowUpdate: {
+      ...acl_attribute,
+      name: "allowUpdate"
+    },
+    allowQuery: {
+      ...acl_attribute,
+      name: "allowQuery"
+    },
+    allowQueryCache: {
+      ...acl_attribute,
+      name: "allowQueryCache"
+    },
+    allowRecursion: {
+      ...acl_attribute,
+      name: "allowRecursion"
+    },
+    allowTransfer: {
+      ...acl_attribute,
+      name: "allowTransfer"
     },
     domains: {
       ...string_set_attribute,
@@ -232,12 +294,6 @@ class bind_group extends Base {
       name: "sharedWith",
       type: bind_group
     },
-    allowedUpdates: {
-      type: bindNetworkAddressTypes,
-      name: "allowedUpdates",
-      collection: true,
-      writable: true
-    },
     notify: { ...boolean_attribute_writable_false, name: "notify" },
     hasCatalog: { ...boolean_attribute_writable_true, name: "hasCatalog" },
     hasReverse: { ...boolean_attribute_writable_false, name: "hasReverse" },
@@ -257,7 +313,7 @@ class bind_group extends Base {
     serial: {
       ...integer_attribute_writable,
       name: "serial",
-      default: Math.ceil(Date.now() / 1000)
+      default: Math.ceil(Date.now() / (1000 * 60)) * 60
     },
     refresh: {
       ...duration_attribute_writable,
@@ -274,7 +330,6 @@ class bind_group extends Base {
   }
 
   foreignDomains = new Set();
-  allowedUpdates = [];
   notify = true;
   hasCatalog = true;
   hasSVRRecords = true;
@@ -296,10 +351,6 @@ class bind_group extends Base {
     return this.sharedWith ? this.sharedWith.order + 1 : 0;
   }
 
-  get service() {
-    return this.owner;
-  }
-
   get soaUpdates() {
     return [this.serial, this.refresh, this.retry, this.expire, this.minimum];
   }
@@ -501,36 +552,6 @@ class bind_group extends Base {
       ))
     );
 
-    return (
-      await Promise.all([
-        this.generateACLs(outputControl),
-        this.generateZoneDefs(outputControl)
-      ])
-    ).find(r => r)
-      ? true
-      : false;
-  }
-
-  async generateACLs(outputControl) {
-    const acls = addressesStatement(
-      `acl ${this.name}`,
-      addresses(this.access, { aggregate: true })
-    );
-
-    if (acls.length) {
-      await writeLines(
-        join(outputControl.dir, "etc/named"),
-        `0-acl-${this.name}.conf`,
-        acls
-      );
-
-      return true;
-    }
-
-    return false;
-  }
-
-  async generateZoneDefs(outputControl) {
     for (const config of this.zoneConfigs.values()) {
       await config.write(outputControl);
     }
@@ -556,9 +577,20 @@ class bind_group extends Base {
  * @returns {(string|string[])[]}
  */
 function addressesStatement(prefix, objects, empty = false, indent = "") {
-  const body = asArray(objects).map(
-    value => `${indent}  ${typeof value === "string" ? value : value.name};`
-  );
+  const body = asArray(objects).map(value => {
+    if (typeof value !== "string") {
+      if (value.name) {
+        value = value.name;
+      } else {
+        value = value.address;
+
+        // console.log(value);
+        //value = [...value];
+      }
+    }
+
+    return `${indent}${value};`;
+  });
 
   if (body.length) {
     return [`${indent}${prefix} {`, body, `${indent}};`];
@@ -579,6 +611,12 @@ export class bind extends CoreService {
       name: "forwarders",
       deferredExpression: true
     },
+    acls: {
+      ...default_collection_attribute_writable,
+      name: "acls",
+      type: bind_acl,
+      backpointer: owner_attribute
+    },
     groups: {
       ...default_collection_attribute_writable,
       name: "groups",
@@ -635,6 +673,7 @@ export class bind extends CoreService {
     addType(this);
   }
 
+  acls = new Map();
   groups = new Map();
 
   get serverType() {
@@ -678,6 +717,11 @@ export class bind extends CoreService {
 
     const outputControl = { packageData, dir, permissions };
 
+    for (const acl of this.acls.values()) {
+      const present = await acl.packageContent(outputControl);
+      hasContent ||= present;
+    }
+
     for (const group of this.groups.values()) {
       const present = await group.packageContent(outputControl);
       hasContent ||= present;

package/src/utils.mjs

@@ -77,7 +77,7 @@ export function sectionLines(sectionName, values) {
   return lines;
 }
 
-export function* setionLinesFromAttributeIterator(properties, separator="=") {
+export function* setionLinesFromAttributeIterator(properties, separator = "=") {
   for (const [name, value, path, attribute] of properties) {
     if (attribute.attributes) {
       yield `[${name}]`;
@@ -96,7 +96,18 @@ export function bridgeToJSON(bridge) {
  * @returns {Array<any>}
  */
 export function asArray(value) {
-  return Array.isArray(value) ? value : value === undefined ? [] : [value];
+  if (value === undefined) {
+    return [];
+  }
+  if (Array.isArray(value)) {
+    return value;
+  }
+
+  if (value instanceof Iterator) {
+    return [...value];
+  }
+
+  return [value];
 }
 
 /**