The IBodyModelValidator does not write to the trace writer, but it should.
The TraceManager should be modified to wrap the IBodyModelValidator to log the Begin and End for the Validate. The End log should report the final result and ModelState. It should also capture and report exceptions like the other trace wrappers.
Without this, users analyzing the trace are unaware which validator was used, what amount of the time was spent in model validation, where it occurred in the pipeline, or what ModelState changes it made.
Comments: We took a look at this and it seems that the full exception details are always provided, and that was the main concern. We are not currently as concerned about additional data not being present.
The TraceManager should be modified to wrap the IBodyModelValidator to log the Begin and End for the Validate. The End log should report the final result and ModelState. It should also capture and report exceptions like the other trace wrappers.
Without this, users analyzing the trace are unaware which validator was used, what amount of the time was spent in model validation, where it occurred in the pipeline, or what ModelState changes it made.
Comments: We took a look at this and it seems that the full exception details are always provided, and that was the main concern. We are not currently as concerned about additional data not being present.