microsoftgraph / microsoftgraph/msgraph-sdk-dotnet

SendMailPostRequestBody defaults SaveToSentItems to false in 6.2.0

Open
#3,167 1 comment 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

status:waiting-for-triage
Dominant language
C#
Stars
789
Forks
264
Avg merge
15h 17m
Merged PRs (30d)
3

Description

Describe the bug

Microsoft.Graph 6.2.0 initializes SendMailPostRequestBody.SaveToSentItems to false.

As a result, applications that do not explicitly assign this optional property serialize "saveToSentItems": false.
Messages are delivered successfully but are not saved in the sender's Sent Items folder.

This conflicts with the Microsoft Graph sendMail API contract, which documents saveToSentItems as optional,
defaulting to true, and says it should be specified only when the desired value is false:

https://learn.microsoft.com/en-us/graph/api/user-sendmail?view=graph-rest-1.0

This behavior differs from Microsoft.Graph 5.104.0, where the generated request-body constructor leaves
SaveToSentItems unset (null) and therefore allows the service default to apply.

Expected behavior

Constructing a request body without explicitly assigning SaveToSentItems should omit the property from the
serialized request:

var requestBody = new SendMailPostRequestBody
{
    Message = message,
};

The service should consequently apply its documented default of true, and the sent message should appear in Sent
Items.

Alternatively, if the SDK assigns a client-side default, it should assign true, not false.

How to reproduce

Using Microsoft.Graph 6.2.0:

using Microsoft.Graph;
using Microsoft.Graph.Me.SendMail;
using Microsoft.Graph.Models;

var message = new Message
{
    Subject = "Sent Items reproduction",
    Body = new ItemBody
    {
        ContentType = BodyType.Text,
        Content = "Test message",
    },
    ToRecipients =
    [
        new Recipient
        {
            EmailAddress = new EmailAddress
            {
                Address = "recipient@example.com",
            },
        },
    ],
};

var requestBody = new SendMailPostRequestBody
{
    Message = message,
};

await graphClient.Me.SendMail.PostAsync(requestBody);

Observe that:

  1. The recipient receives the message.
  2. The message is not present in the authenticated sender's Sent Items folder.
  3. The request body serializes saveToSentItems as false.

The compiled 6.2.0 constructor contains the equivalent of:

SaveToSentItems = false;

The same constructor in Microsoft.Graph 5.104.0 does not initialize this property.

SDK Version

6.2.0

Latest version known to work for scenario above?

5.104.0

Known Workarounds

Explicitly assign SaveToSentItems = true:

var requestBody = new SendMailPostRequestBody
{
    Message = message,
    SaveToSentItems = true,
};
Debug output

Relevant constructor IL from Microsoft.Graph 6.2.0:

ldarg.0
ldc.i4.0
newobj System.Void System.Nullable<System.Boolean>::.ctor(!0)
call System.Void
Microsoft.Graph.Me.SendMail.SendMailPostRequestBody::set_SaveToSentItems(System.Nullable<System.Boolean>)

The generated serializer then writes the property:

call System.Nullable<System.Boolean> SendMailPostRequestBody::get_SaveToSentItems()
callvirt System.Void ISerializationWriter::WriteBoolValue(...)

In Microsoft.Graph 5.104.0, the constructor has no corresponding assignment to SaveToSentItems.

Configuration
  • Microsoft.Graph 6.2.0
  • Microsoft.Graph.Core 4.0.1
  • Microsoft.Kiota.Abstractions 2.0.0
  • Microsoft.Kiota.Serialization.Json 2.0.0
  • .NET 10
  • Windows
  • Delegated authentication
  • Endpoint: POST /me/sendMail
Other information

Microsoft.Graph 6.2.0's release notes mention “generation: update request builders and models,” so this appears to
have been introduced by the model-generation update:

https://github.com/microsoftgraph/msgraph-sdk-dotnet/releases/tag/6.2.0

The REST endpoint itself does not appear to be at fault. Omitting saveToSentItems in a direct REST request uses the
documented service default and saves the message in Sent Items.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the generated SendMailPostRequestBody model and its constructor, then compare the 6.2.0 and 5.104.0 versions described in the issue. Verify serialization for an unset SaveToSentItems value and confirm that the request omits saveToSentItems so the /me/sendMail service default applies.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
api, backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Clearly specified
Newbie friendliness
74/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.