Subject: [netmod] ACL: Issue 5: The TCP flags container is a leaf. This allows only 1 TCP value to be specified per ACE.

From: Sonal Agarwal <[email-masked]>
Date: Fri, 7 Jul 2017 16:00:48 -0700

https://github.com/netmod-wg/acl-model/issues/5

leaf flags {
  type uint16;
  description
    "Also known as Control Bits. Contains 9 1-bit flags.";
}

Further, the TCP flags are not defined in the model.

The improvements to this would be:

   1. The data structure to store the TCP flags should be a leaf list.
   2. The values could be defined to be something like:

identity TCP_FLAGS {
description
"Common TCP flags used in packet header matches";
reference
"IETF RFC 793 - Transmission Control Protocol
IETF RFC 3168 - The Addition of Explicit Congestion
Notification (ECN) to IP";
}

identity TCP_SYN {
base TCP_FLAGS;
description
"TCP SYN flag";
}
...
...

and so on.