Signature validation filter config

Brent Putman putmanb at georgetown.edu
Mon Apr 14 13:14:41 EDT 2014


On 4/14/14 10:21 AM, Rod Widdowson wrote:
>>> 1) The default value for 'requireSignedMetadata' in V2 was FALSE.
>> I think you mean requireValidMetadata.
> No  I really do mean "requireSignedMetadata".  It's an attribute inside
> xsi:type="sec: SignatureValidation".  If it is not specified (or defined to
> be false) then the SignatureValidationFilter will not complain if the
> supplied metadata doesn't have a signature.  With it true
> SignatureValidationFilter complains if the signature fails or if there is no
> signature.  My feeling is that having the default be false opens up to an
> attack in which unsigned metadata is provided as a replacement to signed
> metadata.
>  

Yes, I agree.  I don't have any particular recollection as to why that
defaulted to 'false'.  It might just have been an oversight.  I'm
looking at it right now, so I just go ahead and change the actual filter
to default to 'true'.



>> I'm fine with a subset, but I probably would like to see both cert and
>> public key supported, just because it's gratuitous to even require a cert
>> there.
> Good point.   I'll see about adding both.
>

I can think of at least 2 use cases where the user would want or need to
wire in multiple validation certs/keys: 1)  for the dynamic metadata
resolvers(s), where you really do have multiple concurrent trust
material 2) key rollover, i.e. the metadata publisher announces that
they are switching to a new key on a particular data, and the deployers
want to wire in both keys so the change is transparent. If we think that
those 2 cases are uncommon enough to warrant doing away with Chad's
original plan, then I suppose those people would just go to the full
trust engine option.

If we do go with the attributes option, should we consider making those
XML delimited lists of certs/keys?  Maybe that's too complicated...






More information about the dev mailing list