Improve spec fidelity by creating union of all SignedParts

Author: blaggacao

src/zeep/wsdl/definitions.py

@@ -136,7 +136,7 @@ def __init__(self, wsdl, name, port_name):
         self.wsdl = wsdl
         self._operations = {}
         self.signatures = {
-            "header": [],  # Elements of header, that should be signed
+            "header": [],  # Parts of header, that should be signed
             "body": False,  # If body should be signed
             "everything": False,  # If every header should be signed
         }

src/zeep/wsdl/wsdl.py

@@ -454,6 +454,9 @@ def parse_binding(
                         ),
                         namespaces=NSMAP,
                     )
+                    # Initialize a set to keep track of all unique headers
+                    all_headers = set()
+
                     for sign in signed_parts:
                         if len(sign.getchildren()) == 0:
                             # No children, we should sign everything
@@ -464,11 +467,15 @@ def parse_binding(
                         for child in sign.iterchildren():
                             if len(child.items()) > 0:
                                 # Header ...
-                                part = {attr: value for attr, value in child.items()}
-                                binding.signatures["header"].append(part)
+                                part = frozenset({attr: value for attr, value in child.items()}.items())
+                                all_headers.add(part)
                             elif child.tag.split("}")[-1].lower() == "body":
                                 # Body ...
                                 binding.signatures["body"] = True
+
+                    # If we didn't set "everything" to True, update the headers
+                    if not binding.signatures.get("everything", False):
+                        binding.signatures["header"] = [dict(header) for header in all_headers]
                     logger.debug("Adding binding: %s", binding.name.text)
                     result[binding.name.text] = binding
                     break

src/zeep/wsse/signature.py

@@ -254,18 +254,18 @@ def _signature_prepare(envelope, key, signature_method, digest_method, signature
         if timestamp != None:
             _sign_node(ctx, signature, timestamp, digest_method)
     else:
-        if signatures["body"] or signatures["everything"]:
+        if signatures.get("body") or signatures.get("everything"):
             _sign_node(
                 ctx, signature, envelope.find(QName(soap_env, "Body")), digest_method
             )
         header = get_or_create_header(envelope)
-        if signatures["everything"]:
+        if signatures.get("everything"):
             for node in header.iterchildren():
                 # Everything doesn't mean everything ...
                 if node.nsmap.get(node.prefix) not in OMITTED_HEADERS:
                     _sign_node(ctx, signature, node, digest_method)
         else:
-            for node in signatures["header"]:
+            for node in signatures.get("header", []):
                 _sign_node(
                     ctx,
                     signature,