santuario-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 44206] - KeyInfo does not contain a disg ns declaration when EncryptedKey is martialed
Date Fri, 15 Feb 2008 15:03:09 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=44206>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=44206


sean.mullan@sun.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |WONTFIX




------- Additional Comments From sean.mullan@sun.com  2008-02-15 07:03 -------
(In reply to comment #4)
> I now feel that this behavior is dependent on the XML serializer being used. I
> guess the serializer I am using is not emitting the namespace declaration of a
> namespace-qualified element unless the declaration is added explicitly. Given
> that, this is probably not an important issue and you can close the bug, and
> I'll use the proposed patch at my end. But it probably wouldn't hurt to submit
> the patch as it simply adds the dsig namespace declaration explicitly to the
> <ds:KeyInfo> element, as is done for <ds:Signature> element in the library.

For now, I'm going to close this bug. The problem with the patch is that then
the namespace attribute is always emitted for the KeyInfo element, even when it
isn't necessary. I think a better fix is to add a new constructor or method that
optionally enables this behavior.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

Mime
View raw message