Conversation
- *Fixed:* The `ValueContentResult.TryCreateValueContentResult` would return `NotModified` where the request `ETag` was `null`; this has been corrected to return `OK` with the resulting `value`. - *Fixed:* The `ValueContentResult.TryCreateValueContentResult` now returns `ExtendedStatusCodeResult` versus `StatusCodeResult` as this offers additional capabilities where required. - *Enhancement:* The `ExtendedStatusCodeResult` and `ExtendedContentResult` now implement `IExtendedActionResult` to standardize access to the `BeforeExtension` and `AfterExtension` functions. - *Enhancement:* Added `WebApiParam.CreateActionResult` helper methods to enable execution of the underlying `ValueContentResult.CreateValueContentResult` (which is no longer public as this was always intended as internal only). - *Fixed:* `PostgresDatabase.OnDbException` corrected to use `PostgresException.MessageText` versus `Message` as it does not include the `SQLSTATE` code. - *Enhancement:* Improve debugging insights by adding `ILogger.LogDebug` start/stop/elapsed for the `InvokerArgs`. - *Fixed*: Updated `System.Text.Json` package depenedency to latest (including related); resolve [Microsoft Security Advisory CVE-2024-43485](GHSA-8g4q-xg66-9fp4).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ValueContentResult.TryCreateValueContentResultwould returnNotModifiedwhere the requestETagwasnull; this has been corrected to returnOKwith the resultingvalue.ValueContentResult.TryCreateValueContentResultnow returnsExtendedStatusCodeResultversusStatusCodeResultas this offers additional capabilities where required.ExtendedStatusCodeResultandExtendedContentResultnow implementIExtendedActionResultto standardize access to theBeforeExtensionandAfterExtensionfunctions.WebApiParam.CreateActionResulthelper methods to enable execution of the underlyingValueContentResult.CreateValueContentResult(which is no longer public as this was always intended as internal only).PostgresDatabase.OnDbExceptioncorrected to usePostgresException.MessageTextversusMessageas it does not include theSQLSTATEcode.ILogger.LogDebugstart/stop/elapsed for theInvokerArgs.System.Text.Jsonpackage depenedency to latest (including related); resolve Microsoft Security Advisory CVE-2024-43485.