diff options
| author | William Godbe <wigodbe@microsoft.com> | 2022-08-10 02:21:08 +0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-08-10 02:21:08 +0300 |
| commit | 23cfbbdca163816531952560e12b603a59b156c8 (patch) | |
| tree | 4f4377d8f00da3791950a7525c8c386dfcb355a6 | |
| parent | ef64f61d2991d50af258881603f33d39d5577ac5 (diff) | |
Add comment about QueryString in more placeswtgodbe/QString
| -rw-r--r-- | src/Middleware/HttpLogging/src/HttpLoggingFields.cs | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/src/Middleware/HttpLogging/src/HttpLoggingFields.cs b/src/Middleware/HttpLogging/src/HttpLoggingFields.cs index 82f2cab708..3aa63637b0 100644 --- a/src/Middleware/HttpLogging/src/HttpLoggingFields.cs +++ b/src/Middleware/HttpLogging/src/HttpLoggingFields.cs @@ -138,12 +138,20 @@ public enum HttpLoggingFields : long /// including <see cref="RequestPath"/>, <see cref="RequestProtocol"/>, /// <see cref="RequestMethod"/>, and <see cref="RequestScheme"/>. /// </summary> + /// <remarks> + /// The HTTP Request <see cref="HttpRequest.QueryString"/> is not included with this flag as it may contain private information. + /// If desired, it should be explicitly specified with <see cref="RequestQuery"/>. + /// </remarks> RequestProperties = RequestPath | RequestProtocol | RequestMethod | RequestScheme, /// <summary> /// Flag for logging HTTP Request properties and headers. /// Includes <see cref="RequestProperties"/> and <see cref="RequestHeaders"/> /// </summary> + /// <remarks> + /// The HTTP Request <see cref="HttpRequest.QueryString"/> is not included with this flag as it may contain private information. + /// If desired, it should be explicitly specified with <see cref="RequestQuery"/>. + /// </remarks> RequestPropertiesAndHeaders = RequestProperties | RequestHeaders, /// <summary> |