Closed Bug 550618 Opened 15 years ago Closed 15 years ago

XML-RPC WebService returning wrong dateTime format

Categories

(Bugzilla :: WebService, defect)

3.5.3
defect
Not set
critical

Tracking

()

RESOLVED FIXED
Bugzilla 3.6

People

(Reporter: mkanat, Assigned: mkanat)

References

Details

(Keywords: regression)

Attachments

(1 file)

I forgot that the XML-RPC Specification requires that we send back something that really has nothing to do with ISO-8601: 19980717T14:08:55.
Flags: blocking3.6+
Okay, well, it's not *nothing to do* with ISO-8601, but it's the less-commonly-used format. In any case, we have to fix it. :-|
Attached patch v1Splinter Review
Okay, this fixes it. It's mostly just some re-architecture and then replacing the hyphens in the base implementation with nothing.
Assignee: webservice → mkanat
Status: NEW → ASSIGNED
Attachment #430764 - Flags: review?(dkl)
...sorry :(
Depends on: 545299
Keywords: regression
dkl: This is currently my only blocker on the 3.6rc1 release, so a speedy review would be appreciated.
Blocks: 550732
(In reply to comment #5) > Is this bug responsible for > http://landfill.bugzilla.org/bugzillaqa/show_bug.cgi?id=3171 ? Yes.
Comment on attachment 430764 [details] [diff] [review] v1 Code looks good and returns proper date formats in my testing. r=dkl
Attachment #430764 - Flags: review?(dkl) → review+
Should this also be approved for 3.4?
Flags: approval?
Flags: approval3.6?
(In reply to comment #8) > Should this also be approved for 3.4? No, this is a regression from a patch I landed on 3.6, 3.4 should be good.
Flags: approval?
Flags: approval3.6?
Flags: approval3.6+
Flags: approval+
Thank you for the fast review, dkl! I know that it's the weekend, so I really appreciate it. Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/ modified Bugzilla/WebService.pm modified Bugzilla/WebService/Server.pm Committed revision 7057. modified Bugzilla/WebService.pm modified Bugzilla/WebService/Server.pm Committed revision 7024.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: