[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]
Subject: RE: [xacml-comment] XACML 2.0: Small error in dayTimeDuration
Why this restriction? As I understand the data type is a "subset" of the xs:duration that is a duration, with the meaning of a time-span. The xs:duration part of day is specified as "nD the number of days" (from http://www.w3.org/TR/xmlschema-2/#duration). In my opinion this means that it can be a number of arbitrary length. Help me out if I am wrong. Regards, Florian -----Original Message----- From: Bill Parducci [mailto:bill@parducci.net] Sent: Dienstag, 29. September 2009 19:06 To: Florian Huonder Subject: Re: [xacml-comment] XACML 2.0: Small error in dayTimeDuration Ok, makes sense. Wouldn't this match -P100D10H3M12S as well? It is kinda messy but I think this is the more precise match: [-]?P(1|0){0,1}(\p{Nd} ... What do you think? thanks b On Sep 29, 2009, at 9:01 AM, Florian Huonder wrote: > 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 > > > > -- > This publicly archived list offers a means to provide input to the > OASIS eXtensible Access Control Markup Language (XACML) TC. > > In order to verify user consent to the Feedback License terms and > to minimize spam in the list archive, subscription is required > before posting. > > Subscribe: xacml-comment-subscribe@lists.oasis-open.org > Unsubscribe: xacml-comment-unsubscribe@lists.oasis-open.org > List help: xacml-comment-help@lists.oasis-open.org > List archive: http://lists.oasis-open.org/archives/xacml-comment/ > Feedback License: http://www.oasis-open.org/who/ipr/feedback_license.pdf > List Guidelines: http://www.oasis-open.org/maillists/guidelines.php > Committee: http://www.oasis-open.org/committees/tc_home.php?wg_abbrev=xacml >
[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]