GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Incorrect Serialization Format for defaultDuration in BookingService · Issue #1262 · microsoftgraph/msgraph-sdk-python

Incorrect Serialization Format for defaultDuration in BookingService #1262

New issue
Copy link
New issue
Copy link
Open
Open
Incorrect Serialization Format for defaultDuration in BookingService#1262
Copy link
Assignees
ramsessanchez
Labels
type:bugA broken experienceA broken experience

Description

@LeoGCode
Leonel Guerrero (LeoGCode)
opened on Jun 16, 2025
Issue body actions

Describe the bug

The defaultDuration field in the BookingService model is being serialized incorrectly due to the behavior of the write_timedelta_value method in the json_serialization_writer.py module.

According to the Microsoft Graph API documentation for BookingService, the defaultDuration field must be serialized as an ISO 8601 duration, e.g., P11D23H59M59.999999999999S.

However, the SDK currently serializes Python timedelta objects using the default str(timedelta) representation, which does not conform to ISO 8601. This results in requests being sent with incorrectly formatted durations and leading to 400 Graph API errors.

Relevant Code:

File: msgraph/generated/models/booking_service.py

writer.write_timedelta_value("defaultDuration", self.default_duration)

File: kiota_serialization_json/json_serialization_writer.py

def write_timedelta_value(self, key: Optional[str], value: Optional[timedelta]) -> None:
    if isinstance(value, timedelta):
        if key:
            self.writer[key] = str(value)  # <-- Incorrect format
        else:
            self.value = str(value)

Expected behavior

write_timedelta_value should serialize timedelta objects to valid ISO 8601 duration strings (e.g., P1DT2H3M4S), not the default str(timedelta) representation (1 day, 2:03:04).

How to reproduce

Create a BookingService object with a default_duration = timedelta(days=1, hours=2, minutes=3, seconds=4).

Serialize the object.

Observe that the serialized value for defaultDuration is "1 day, 2:03:04" instead of "P1DT2H3M4S".

SDK Version

1.33.0

Latest version known to work for scenario above?

No response

Known Workarounds

Currently manually patching the serialize method to use correct serialization

from datetime import timedelta

from isodate import duration_isoformat
from kiota_abstractions.serialization.serialization_writer import SerializationWriter
from msgraph.generated.models.booking_service import BookingService


def patch_booking_service(booking_service: BookingService) -> None:
    old_serializer = booking_service.serialize

    def booking_service_serializer_patch(writer: SerializationWriter) -> None:
        old_serializer(writer)
        if isinstance(booking_service.default_duration, timedelta):
            writer.writer["defaultDuration"] = duration_isoformat(booking_service.default_duration) 

    booking_service.serialize = booking_service_serializer_patch 

Debug output

Click to expand log

MainError(additional_data={}, code='UnknownError', details=None, inner_error=InnerError(additional_data={}, client_request_id='08e4b63d-43c0-42aa-a8ed-c42fc75cbf2a', date=datetime.datetime(2025, 6, 16, 20, 47, 57), odata_type=None, request_id='3e28d869-8972-4636-a6f6-6cc0a126bee5'), message='{"type":"https://tools.ietf.org/html/rfc9110#section-15.5.1","title":"One or more validation errors occurred.","status":400,"errors":{"":["The input was not valid."]},"traceId":"00-3e28d86989724636a6f66cc0a126bee5-3b0ff72f47b6692f-01"}', target=None)

Configuration

SDK Version: Latest (as of writing)

Python Version: 3.11+

Endpoint: /solutions/bookingBusinesses/{id}/services

Other information

Proposed Fix
Add a new utility function for ISO 8601 serialization
File: kiota_serialization_json/json_serialization_writer.py:

from isodate import duration_isoformat

def write_iso8601_duration(self, key: Optional[str], value: Optional[timedelta]) -> None:
    """
    Writes a timedelta as an ISO 8601 duration string (e.g., P1DT2H3M4S).
    """
    if isinstance(value, timedelta):
        duration = duration_isoformat(value)
        if key:
            self.writer[key] = duration
        else:
            self.value = duration

Then update BookingService to call this new method explicitly:

File: msgraph/generated/models/booking_service.py:

@dataclass
class BookingService(Entity, Parsable):
    ...
    def serialize(self, writer: SerializationWriter) -> None:
        ...
        # Use the custom method for ISO 8601 durations
        if hasattr(writer, "write_iso8601_duration"):
            writer.write_iso8601_duration("defaultDuration", self.default_duration)
        else:
            writer.write_timedelta_value("defaultDuration", self.default_duration)  # Fallback

Activity

  1. Leonel Guerrero (LeoGCode)
    added
    status:waiting-for-triageAn issue that is yet to be reviewed or assignedAn issue that is yet to be reviewed or assigned
    type:bugA broken experienceA broken experience
    on Jun 16, 2025
  2. Luis Ramos (lramosvea)
    removed
    status:waiting-for-triageAn issue that is yet to be reviewed or assignedAn issue that is yet to be reviewed or assigned
    on May 18, 2026
  3. Ramses Sanchez-Hernandez (ramsessanchez)
    self-assigned this
    on May 18, 2026
  4. ramsessanchez commented on May 18, 2026

    @ramsessanchez
    Ramses Sanchez-Hernandez (ramsessanchez)
    on May 18, 2026
    Contributor
    More actions

    Similar too already fixed issue: microsoft/kiota-dotnet#30
    We will need to ensure this is also fixed for this sdk

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

  • @ramsessanchez
    Ramses Sanchez-Hernandez (ramsessanchez)

Labels

type:bugA broken experienceA broken experience

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

    You are visitor 01402503 — GeezerVison does not store the pages you look up.
    Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.