Tagging subscribers to this area:@tommcdon

Learn Build status updates of commita78d259:

✅ Validation status: passed

For more details, please refer to thebuild report.

For any questions, please:

• Try searching the learn.microsoft.comcontributor guides
• Post your question in theLearn support channel

@tommcdon Could you review?

.NET is usingReportEvent, and according to the Windows documentation forReportEvent, our existing documentation is correct:

https://learn.microsoft.com/windows/win32/api/winbase/nf-winbase-reporteventw

[in] lpStringsA pointer to a buffer containing an array of null-terminated strings that are merged into the message before Event Viewer displays the string to the user. This parameter must be a valid pointer (or NULL), even if wNumStrings is zero. Each string is limited to 31,839 characters.Prior to Windows Vista:  Each string is limited to 32K characters.

The .NET code that callsReportEvent is checking 32K-2 characters:

https://github.com/dotnet/runtime/blob/c1a9f26efa4fcf2e3fdcd8557da19d358f51eb00/src/libraries/System.Diagnostics.EventLog/src/System/Diagnostics/EventLogInternal.cs#L1354-L1357

// make sure the strings aren't too long.  MSDN says each string has a limit of 32k (32768) characters, but// experimentation shows that it doesn't like anything larger than 32766if(strings[i].Length>32766)thrownewArgumentException(SR.LogEntryTooLong);

Therefore, the error would likely be coming from Windows and not .NET for string lengths <= 32766 characters. Given that .NET has no control over the version of Windows and the various character limits it is imposing, I suggest we remove mentions of specific character limits under 32766 characters.

Also I would expect that if the string length is <= 32766 and the API call fails due to a restriction from Windows, we would get a Win32Exception and not an ArgumentException.@0xced can you confirm?

If yes, then I suggest changing the text to something like:
The message string is longer than 32,766 bytes.

@0xced Do you plan to address the requested changes?