RE: [xacml-comment] XACML 2.0: Small error in dayTimeDuration

[Florian Huonder <fhuonder@herasaf.org>]

Hi Bill,

I used the following implementation of the regex (in Java):
http://fisheye.herasaf.org/browse/HERAS-AF_Repository/herasaf-xacml/trunk/he
rasaf-xacml-core/src/main/java/org/herasaf/xacml/core/types/DayTimeDuration.
java?r=90
from our XACML Implementation (HERAS-AF). The regex-pattern is on line 49.

If you let this run with e.g. the following example it works:
-P1D10H3M12S

With the following example (day is 2 digits) it does not work:
-P10D10H3M12S

Regards,
Florian


-----Original Message-----
From: bill parducci [mailto:bill@parducci.net] 
Sent: Dienstag, 29. September 2009 16:34
To: Florian Huonder
Cc: xacml-comment@lists.oasis-open.org
Subject: Re: [xacml-comment] XACML 2.0: Small error in dayTimeDuration

Hi Florian,

Can you post the sample datetime data you used to test this working  
and non-working cases?

thanks

b

On Sep 29, 2009, at 4:50 AM, Florian Huonder wrote:

> Hi all,
>
> In XACML 2.0 the definition of the urn:oasis:names:tc:xacml:2.0:data- 
> type:dayTimeDuration contains a small error.
> In my opinion the regular expression does only allow one digit for  
> the day-part.
>
> Currently the regex looks like this:
> [-]?P(\p{Nd}D(T(\p{Nd}+(H(\p{Nd}+(M(\p{Nd}+(\.\p{Nd}*)?S
>             |\.\p{Nd}+S)?|(\.\p{Nd}*)?S)|(\.\p{Nd}*)?S)?|M(\p{Nd}+
>             (\.\p{Nd}*)?S|\.\p{Nd}+S)?|(\.\p{Nd}*)?S)|\.\p{Nd}+S))?
>             |T(\p{Nd}+(H(\p{Nd}+(M(\p{Nd}+(\.\p{Nd}*)?S|\.\p{Nd}+S)?
>             |(\.\p{Nd}*)?S)|(\.\p{Nd}*)?S)?|M(\p{Nd}+(\.\p{Nd}*)?S|\. 
> \p{Nd}+S)?
>             |(\.\p{Nd}*)?S)|\.\p{Nd}+S))
>
> In my opinion it should look like:
> [-]?P(\p{Nd}+D(T(\p{Nd}+(H(\p{Nd}+(M(\p{Nd}+(\.\p{Nd}*)?S
>             |\.\p{Nd}+S)?|(\.\p{Nd}*)?S)|(\.\p{Nd}*)?S)?|M(\p{Nd}+
>             (\.\p{Nd}*)?S|\.\p{Nd}+S)?|(\.\p{Nd}*)?S)|\.\p{Nd}+S))?
>             |T(\p{Nd}+(H(\p{Nd}+(M(\p{Nd}+(\.\p{Nd}*)?S|\.\p{Nd}+S)?
>             |(\.\p{Nd}*)?S)|(\.\p{Nd}*)?S)?|M(\p{Nd}+(\.\p{Nd}*)?S|\. 
> \p{Nd}+S)?
>             |(\.\p{Nd}*)?S)|\.\p{Nd}+S))
>
> (I added a + at the 13. position).
>
>
> By the way, in this section the titles are mixed up, Duration in  
> years and months <-> Durations in days and times.
>
> I don't know how relevant this is for the TC because the most work  
> is put into XACML 3.0 I think.
>
> Regards,
> Florian