Skip to content

Check for deleted status on Search Param update #4883

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Mar 24, 2025
Merged

Check for deleted status on Search Param update #4883

merged 1 commit into from
Mar 24, 2025

Conversation

@feordin
Copy link
Contributor

@feordin feordin commented Mar 24, 2025 •
edited by azure-boards bot
Loading

Description

Adds a check if the previous version of the Search Parameter was already deleted.

Related issues

Addresses [issue AB#140720].

Testing

Manual testing was applied.

FHIR Team Checklist

  • Update the title of the PR to be succinct and less than 65 characters
  • Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
  • Tag the PR with the type of update: Bug, Build, Dependencies, Enhancement, New-Feature or Documentation
  • Tag the PR with Open source, Azure API for FHIR (CosmosDB or common code) or Azure Healthcare APIs (SQL or common code) to specify where this change is intended to be released.
  • Tag the PR with Schema Version backward compatible or Schema Version backward incompatible or Schema Version unchanged if this adds or updates Sql script which is/is not backward compatible with the code.
  • When changing or adding behavior, if your code modifies the system design or changes design assumptions, please create and include an ADR.
  • CI is green before merge Build Status
  • Review squash-merge requirements

Semver Change (docs)

Patch|Skip|Feature|Breaking (reason)

@feordin feordin added Bug Bug bug bug. Azure API for FHIR Label denotes that the issue or PR is relevant to the Azure API for FHIR Azure Healthcare APIs Label denotes that the issue or PR is relevant to the FHIR service in the Azure Healthcare APIs Schema Version unchanged labels Mar 24, 2025
@feordin feordin added this to the 2Wk13 milestone Mar 24, 2025
@feordin feordin requested a review from a team as a code owner March 24, 2025 17:46
@feordin feordin changed the title When performing PUT for search parameter, we missed a check for deleted status Check for deleted status on Search Param update Mar 24, 2025
@feordin feordin enabled auto-merge (squash) March 24, 2025 18:05
github-advanced-security[bot]
github-advanced-security bot found potential problems Mar 24, 2025
View reviewed changes
...crosoft.Health.Fhir.Core/Features/Search/Parameters/CreateOrUpdateSearchParameterBehavior.cs
var resourceKey = new ResourceKey(request.Resource.InstanceType, request.Resource.Id, request.Resource.VersionId);
ResourceWrapper prevSearchParamResource = await _fhirDataStore.GetAsync(resourceKey, cancellationToken);
if (prevSearchParamResource != null)
if (prevSearchParamResource != null && prevSearchParamResource.IsDeleted == false)

Check notice

Code scanning / CodeQL

Unnecessarily complex Boolean expression Note

The expression 'A == false' can be simplified to '!A'.

Copilot Autofix

AI 8 months ago

To fix the problem, we need to simplify the Boolean expression prevSearchParamResource.IsDeleted == false to !prevSearchParamResource.IsDeleted. This change will make the code more readable and maintainable without altering its functionality.

  • Locate the expression prevSearchParamResource.IsDeleted == false in the file src/Microsoft.Health.Fhir.Core/Features/Search/Parameters/CreateOrUpdateSearchParameterBehavior.cs.
  • Replace the expression with !prevSearchParamResource.IsDeleted.
Suggested changeset 1
src/Microsoft.Health.Fhir.Core/Features/Search/Parameters/CreateOrUpdateSearchParameterBehavior.cs

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/src/Microsoft.Health.Fhir.Core/Features/Search/Parameters/CreateOrUpdateSearchParameterBehavior.cs b/src/Microsoft.Health.Fhir.Core/Features/Search/Parameters/CreateOrUpdateSearchParameterBehavior.cs
--- a/src/Microsoft.Health.Fhir.Core/Features/Search/Parameters/CreateOrUpdateSearchParameterBehavior.cs
+++ b/src/Microsoft.Health.Fhir.Core/Features/Search/Parameters/CreateOrUpdateSearchParameterBehavior.cs
@@ -54,3 +54,3 @@
                 ResourceWrapper prevSearchParamResource = await _fhirDataStore.GetAsync(resourceKey, cancellationToken);
-                if (prevSearchParamResource != null && prevSearchParamResource.IsDeleted == false)
+                if (prevSearchParamResource != null && !prevSearchParamResource.IsDeleted)
                 {
EOF
@@ -54,3 +54,3 @@
ResourceWrapper prevSearchParamResource = await _fhirDataStore.GetAsync(resourceKey, cancellationToken);
if (prevSearchParamResource != null && prevSearchParamResource.IsDeleted == false)
if (prevSearchParamResource != null && !prevSearchParamResource.IsDeleted)
{
Copilot is powered by AI and may make mistakes. Always verify output.
@feordin feordin merged commit 698a1fb into main Mar 24, 2025
47 of 58 checks passed
@feordin feordin deleted the personal/jaerwin/update-after-delete-search-param branch March 24, 2025 19:47
paorodma-ms added a commit that referenced this pull request Apr 2, 2025
@paorodma-ms @brendankowitz
@LTA-Thinking @mikaelweave @rajithaalurims @rbans96 @PTaladay @feordin @tarunmathew12 @SergeyGaluzo @v-ajajvanu @dependabot @fhibf @abiisnn
Update raw resource feature branch (#4897)
a9ab0ca 
* Fix merge error in Directory.Packages.props (#4795)

+semver: skip

* Add SQL performance option (#4785)

* Add additional error handling to CosmosHealthCheck (#4781)

* Added additional error handling to CosmosHealthchecks

* Added tests

* fix code scanning error in tests

* make code easier to review

* add test descriptions back to code

* removed unnecessary exception logging

* Moved logging of CosmosDiagnostic to the property bag

* fix code scanning error

* ConnectionMode is fetched from configuration instead of using Direct mode always (#4807)

* Fix instance availability errors on schema upgrade (#4803)

* Fix: Date search doesn't match ranges properly (#4762)

* Fix to allow date time to be searched against a resource with a span and if included in the span that resource will be returned.

* Updated tests based on recent changes

* Updated test fixtures. Removed temporary comments.

* Updated test results to match new expectations. Simplified logic for this particular case and addressed pr comments.

* Fixed unit test after recent changes.

* Updated more tetst based on recent changes.

* Adds ADR readme and Template (#4805)

* Added in check for reindex job in progress. Added unit test to verify. (#4813)

* add cosmos direct code configuration options (#4816)

* Update to Firely 5.11.3, add binary resource test (#4814)

Update FhirService to use Firely 5.11.3 SDK

Refs AB#135366

* Cosmos Direct Mode - Fix Rediscovery Default (#4817)

* add cosmos direct code configuration options

* keep existing behavior for Cosmos Direct rediscovery

* Delete docs/rest/pers-env.http

* Delete docs/rest/export-test.http

* Add support for include to delete requests (#4811)

* Deployment template can specify registryName (#4809)

* Catch shutdown cancellation exceptions for conformance provider (#4825)

* Catch shutdown cancellation exceptions for conformance provider

* reenable disposing semaphores

* Simplify disposal code

* undo semaphore slim disposal changes

* Fixing a missing retry in CosmosFhirDataStore. (#4831)

* Remove resource id from hash (#4832)

Remove resource id from hash

Refs AB#135263

* Add exponential retry and jitter to importer (#4840)

* Enable schema compare test locally and make normalization generic (#4845)

* Reviving schema upgrade test for local

* Revive schema comparison test and make SQL normalization generic

* usings

* fixed var check

* return

* Corections after merge

* Remove legacy export job worker (#4838)

* Remove LegacyExportJobWorker #4750

* Remove UseQueueClientJobs configuration since it is the default now, no other possibility. Remove unused variables from CosmosFhirOperationDataStore

* Remove classes and stored procedures from Cosmos DB related to LegacyExportJob. Modified unit test in CosmosDBInitializationTests to appropriately check for all incoming list of sp's

* Readded necessary legacy code to be able to query a legacy export job.

* Remove commented code from test

* Add schema version 86, removing unused table ExportJob

* Documenting the steps for connecting to SQL database (#4619)

Co-authored-by: Ajaj Vanu <[email protected]>

* Increase max valueset size (#4850)

* 86 -> 87

* insert version 87

* Fixing an issue of a next link leading to an empty page. (#4847)

* insert correct schema version 86 (#4853)

* remove commented out text

* Remove forwarded headers and add classes to use antissrf in wp (#4849)

* Add exception/configuration class to use antissrf in managed solution

* Removed unused headers

* Added bundle

* bundle size

* Fixing schema version (#4862)

* disabling watchdogs with raw resource split

* Added cleanup on failed transactions

* tests twist

* delete from parameter

* tests

* Bump coverlet.collector from 6.0.2 to 6.0.4 (#4802)

Bumps [coverlet.collector](https://github.com/coverlet-coverage/coverlet) from 6.0.2 to 6.0.4.
- [Release notes](https://github.com/coverlet-coverage/coverlet/releases)
- [Commits](coverlet-coverage/coverlet@v6.0.2...v6.0.4)

---
updated-dependencies:
- dependency-name: coverlet.collector
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump Newtonsoft.Json and Microsoft.NET.Test.Sdk (#4758)

Bumps [Newtonsoft.Json](https://github.com/JamesNK/Newtonsoft.Json) and [Microsoft.NET.Test.Sdk](https://github.com/microsoft/vstest). These dependencies needed to be updated together.

Updates `Newtonsoft.Json` from 13.0.3 to 13.0.1
- [Release notes](https://github.com/JamesNK/Newtonsoft.Json/releases)
- [Commits](JamesNK/Newtonsoft.Json@13.0.3...13.0.1)

Updates `Microsoft.NET.Test.Sdk` from 17.10.0 to 17.12.0
- [Release notes](https://github.com/microsoft/vstest/releases)
- [Changelog](https://github.com/microsoft/vstest/blob/main/docs/releases.md)
- [Commits](microsoft/vstest@v17.10.0...v17.12.0)

---
updated-dependencies:
- dependency-name: Newtonsoft.Json
  dependency-type: direct:production
  update-type: version-update:semver-patch
- dependency-name: Microsoft.NET.Test.Sdk
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* [Bundle] Transactions with single record are parallel (#4858)

* Add logic to handle transactions with a single record as parallel

* Adding new tests to validate the bundle response.

* New test to validate if bundles submitted as transactions with a single record are processed in parallel.

* Testing new Clone behavior.

* Undo global.json changes

* Add a better handling to C# transaction failures.

* Adding description to new exception extention.

* Always use external buildx image (#4864)

* always use external buildx image

* updated builder logic

* undo unneeded change

* added comment to docker

* updating buildx create to remove platform (unneeded)

* fix typo

* forcing builder instance

* fix typo in buildx

* add qemu setup

* formatting

* Use mirror for buildx qemu multi-arch builder (#4870)

* always use external buildx image

* updated builder logic

* undo unneeded change

* added comment to docker

* updating buildx create to remove platform (unneeded)

* fix typo

* forcing builder instance

* fix typo in buildx

* add qemu setup

* formatting

* add mirror for qemu builder image

* fix comment

* change docker run qemu to use mirror

* Implementing $includes operation with paging.  (#4810)

* Implementing $include operation with paging.

* Fixing some UT failures.

* Adding some UT/E2E test for $includes operation.

* Fixing some test failures.

* Fixing test failures.

* Addressing RB's comments.

* Removing a redundant operation outcome for truncated include resources.

* Fixing UT

* Adding feature flag for $includes operation

* Updating PR.

* Adding a feature flag for $includes to appsettings.json.

* Adding the first batch of included resources to the original bundle response when SupportsIncludes is true.

* Addressing Brendan's comment.

* Removing a duplicate of setting up the search expressions in SearchImpl/SearchIncludesImpl.

* Enabling $includes operation by default.

* Addressing Brendan and RB's comments.

* Addressing Brendan and RB's comments pt.2

* Trimming included resources only when the SupportsIncludes is false.

* Fixing one failed SearchOptionsFactoryTests.

* Fixing an issue of metadata having $includes details for CosmosDb.

* Adds ADR

* Fixed limit text

---------

Co-authored-by: Brendan Kowitz <[email protected]>

* Bump dotnet/aspnet in /build/docker (#4860)

Bumps dotnet/aspnet from 9.0.1-azurelinux3.0 to 9.0.3-azurelinux3.0.

---
updated-dependencies:
- dependency-name: dotnet/aspnet
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump System.Text.Encodings.Web (#4822)

Bumps [Microsoft.Extensions.Configuration](https://github.com/dotnet/runtime), [Microsoft.Extensions.Configuration.Json](https://github.com/dotnet/runtime) and [System.Text.Encodings.Web](https://github.com/dotnet/runtime). These dependencies needed to be updated together.

Updates `Microsoft.Extensions.Configuration` from 8.0.0 to 9.0.2
- [Release notes](https://github.com/dotnet/runtime/releases)
- [Commits](dotnet/runtime@v8.0.0...v9.0.2)

Updates `Microsoft.Extensions.Configuration.Json` from 8.0.0 to 9.0.2
- [Release notes](https://github.com/dotnet/runtime/releases)
- [Commits](dotnet/runtime@v8.0.0...v9.0.2)

Updates `System.Text.Encodings.Web` from 9.0.1 to 9.0.2
- [Release notes](https://github.com/dotnet/runtime/releases)
- [Commits](dotnet/runtime@v9.0.1...v9.0.2)

---
updated-dependencies:
- dependency-name: Microsoft.Extensions.Configuration
  dependency-type: direct:production
  update-type: version-update:semver-major
- dependency-name: Microsoft.Extensions.Configuration.Json
  dependency-type: direct:production
  update-type: version-update:semver-major
- dependency-name: System.Text.Encodings.Web
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump System.Security.Cryptography.Pkcs and System.Security.Cryptography.Xml (#4826)

Bumps [System.Security.Cryptography.Pkcs](https://github.com/dotnet/runtime) and [System.Security.Cryptography.Xml](https://github.com/dotnet/runtime). These dependencies needed to be updated together.

Updates `System.Security.Cryptography.Pkcs` from 8.0.1 to 9.0.2
- [Release notes](https://github.com/dotnet/runtime/releases)
- [Commits](dotnet/runtime@v8.0.1...v9.0.2)

Updates `System.Security.Cryptography.Xml` from 8.0.2 to 9.0.2
- [Release notes](https://github.com/dotnet/runtime/releases)
- [Commits](dotnet/runtime@v8.0.2...v9.0.2)

---
updated-dependencies:
- dependency-name: System.Security.Cryptography.Pkcs
  dependency-type: direct:production
  update-type: version-update:semver-major
- dependency-name: System.Security.Cryptography.Xml
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump System.IdentityModel.Tokens.Jwt from 8.1.2 to 8.3.0 (#4748)

* Bump System.IdentityModel.Tokens.Jwt from 8.1.2 to 8.3.0

Bumps [System.IdentityModel.Tokens.Jwt](https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet) from 8.1.2 to 8.3.0.
- [Release notes](https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/releases)
- [Changelog](https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/blob/dev/CHANGELOG.md)
- [Commits](AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet@8.1.2...8.3.0)

---
updated-dependencies:
- dependency-name: System.IdentityModel.Tokens.Jwt
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* Remove System.Data.SqlClient package version.

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Mikael Weaver <[email protected]>

* Bump dotnet/sdk in /build/docker (#4861)

Bumps dotnet/sdk from 9.0.102-azurelinux3.0 to 9.0.201-azurelinux3.0.

---
updated-dependencies:
- dependency-name: dotnet/sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump System.Collections.Immutable (#4774)

* Bump System.Collections.Immutable from 8.0.0 to 9.0.1

Bumps [System.Collections.Immutable](https://github.com/dotnet/runtime) from 8.0.0 to 9.0.1.
- [Release notes](https://github.com/dotnet/runtime/releases)
- [Commits](dotnet/runtime@v8.0.0...v9.0.1)

---
updated-dependencies:
- dependency-name: System.Collections.Immutable
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

* Update System.Collections.Immutable version to SdkPackageVersion

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Mikael Weaver <[email protected]>

* Bump System.Text.Encodings.Web (#4827)

Bumps [Microsoft.Extensions.Configuration](https://github.com/dotnet/runtime), [Microsoft.Extensions.Logging](https://github.com/dotnet/runtime) and [Microsoft.Extensions.Logging.Console](https://github.com/dotnet/runtime). These dependencies needed to be updated together.

Updates `Microsoft.Extensions.Configuration` from 8.0.0 to 9.0.2
- [Release notes](https://github.com/dotnet/runtime/releases)
- [Commits](dotnet/runtime@v8.0.0...v9.0.2)

Updates `Microsoft.Extensions.Logging` from 8.0.1 to 9.0.2
- [Release notes](https://github.com/dotnet/runtime/releases)
- [Commits](dotnet/runtime@v8.0.1...v9.0.2)

Updates `Microsoft.Extensions.Logging.Console` from 8.0.0 to 9.0.2
- [Release notes](https://github.com/dotnet/runtime/releases)
- [Commits](dotnet/runtime@v8.0.0...v9.0.2)

---
updated-dependencies:
- dependency-name: Microsoft.Extensions.Configuration
  dependency-type: direct:production
  update-type: version-update:semver-major
- dependency-name: Microsoft.Extensions.Logging
  dependency-type: direct:production
  update-type: version-update:semver-major
- dependency-name: Microsoft.Extensions.Logging.Console
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* bump Polly from 8.4.2 to 8.5.2 (#4837)

---
updated-dependencies:
- dependency-name: Polly
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump xunit.extensibility.core from 2.9.0 to 2.9.3 (#4757)

* Bump xunit.extensibility.core from 2.9.0 to 2.9.3

Bumps [xunit.extensibility.core](https://github.com/xunit/xunit) from 2.9.0 to 2.9.2.
- [Commits](xunit/xunit@v2-2.9.0...v2-2.9.2)

---
updated-dependencies:
- dependency-name: xunit.extensibility.core
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Update all xunit packages together

* Update xunit package versions to 2.9.3

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Mikael Weaver <[email protected]>

* Bump Azure.Extensions.AspNetCore.Configuration.Secrets (#4866)

Bumps [Azure.Extensions.AspNetCore.Configuration.Secrets](https://github.com/Azure/azure-sdk-for-net) from 1.3.2 to 1.4.0.
- [Release notes](https://github.com/Azure/azure-sdk-for-net/releases)
- [Commits](Azure/azure-sdk-for-net@Azure.Extensions.AspNetCore.Configuration.Secrets_1.3.2...Azure.Extensions.AspNetCore.Configuration.Secrets_1.4.0)

---
updated-dependencies:
- dependency-name: Azure.Extensions.AspNetCore.Configuration.Secrets
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump HealthcareSharedPackageVersion (#4829)

* bump Microsoft.Extensions.Configuration.Binder and Microsoft.Extensions.Options.ConfigurationExtensions

---
updated-dependencies:
- dependency-name: Microsoft.Extensions.Configuration.Binder
  dependency-type: direct:production
  update-type: version-update:semver-major
- dependency-name: Microsoft.Extensions.Options.ConfigurationExtensions
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

* update to latest shared component version

* update packages to make build work

* simplified package version changes

* remove global floating packages

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Mikael Weaver <[email protected]>

* Return 400 when too many parameters (#4855)

* Adds ADR proposals (#4871)

* Adds ADR readme

* Date based adr number

* Adds ADR for $include operation

* Moves to proposals folder

* Adds ADRs for ResourceIdMap and Resource table split

* Include on delete

* database adrs updated

* Updates to schema update ADRs

* Update adr-2501-Bundle-include-operation.md

* moves images

---------

Co-authored-by: Jared Erwin <[email protected]>

* use variable for .net sdk packages (#4879)

* Bump dotnet/sdk in /build/docker (#4875)

Bumps dotnet/sdk from 9.0.201-azurelinux3.0 to 9.0.202-azurelinux3.0.

---
updated-dependencies:
- dependency-name: dotnet/sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump AngleSharp from 1.1.2 to 1.2.0 (#4865)

Bumps [AngleSharp](https://github.com/AngleSharp/AngleSharp) from 1.1.2 to 1.2.0.
- [Release notes](https://github.com/AngleSharp/AngleSharp/releases)
- [Changelog](https://github.com/AngleSharp/AngleSharp/blob/devel/CHANGELOG.md)
- [Commits](AngleSharp/AngleSharp@1.1.2...1.2.0)

---
updated-dependencies:
- dependency-name: AngleSharp
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump xunit.runner.visualstudio from 2.8.2 to 3.0.2 (#4836)

---
updated-dependencies:
- dependency-name: xunit.runner.visualstudio
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump Hl7.Fhir.STU3 from 5.11.3 to 5.11.4 (#4833)

Bumps [Hl7.Fhir.STU3](https://github.com/FirelyTeam/firely-net-sdk) from 5.11.3 to 5.11.4.
- [Release notes](https://github.com/FirelyTeam/firely-net-sdk/releases)
- [Changelog](https://github.com/FirelyTeam/firely-net-sdk/blob/develop/release-notes.md)
- [Commits](FirelyTeam/firely-net-sdk@v5.11.3...v5.11.4)

---
updated-dependencies:
- dependency-name: Hl7.Fhir.STU3
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump Microsoft.Azure.Cosmos (#4877)

Bumps [Microsoft.Azure.Cosmos](https://github.com/Azure/azure-cosmos-dotnet-v3) and [System.Text.Encodings.Web](https://github.com/dotnet/runtime). These dependencies needed to be updated together.

Updates `Microsoft.Azure.Cosmos` from 3.45.2 to 3.47.2
- [Release notes](https://github.com/Azure/azure-cosmos-dotnet-v3/releases)
- [Changelog](https://github.com/Azure/azure-cosmos-dotnet-v3/blob/master/changelog.md)
- [Commits](Azure/azure-cosmos-dotnet-v3@3.45.2...3.47.2)

Updates `System.Text.Encodings.Web` from 9.0.2 to 6.0.0
- [Release notes](https://github.com/dotnet/runtime/releases)
- [Commits](dotnet/runtime@v9.0.2...v6.0.0)

---
updated-dependencies:
- dependency-name: Microsoft.Azure.Cosmos
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: System.Text.Encodings.Web
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add deleted status check (#4883)

Add deleted status check (#4883)

Refs AB#140720

* Update schema to 88

* Read blob async

* Fix schema to 87, update unit tests

* Add includes continuation token

* Update Azure.Storage.Blob version

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Brendan Kowitz <[email protected]>
Co-authored-by: Robert Johnson <[email protected]>
Co-authored-by: Mikael Weaver <[email protected]>
Co-authored-by: rajithaalurims <[email protected]>
Co-authored-by: Richa Bansal <[email protected]>
Co-authored-by: Paul Taladay <[email protected]>
Co-authored-by: Jared Erwin <[email protected]>
Co-authored-by: tarunmathew12 <[email protected]>
Co-authored-by: SergeyGaluzo <[email protected]>
Co-authored-by: Sergey Galuzo <[email protected]>
Co-authored-by: v-ajajvanu <[email protected]>
Co-authored-by: Ajaj Vanu <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Fernando Henrique Inocêncio Borba Ferreira <[email protected]>
Co-authored-by: Mikael Weaver <[email protected]>
Co-authored-by: Abigail Nicolas <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rbans96 rbans96 rbans96 approved these changes

Assignees

No one assigned

Labels

Azure API for FHIR Label denotes that the issue or PR is relevant to the Azure API for FHIR Azure Healthcare APIs Label denotes that the issue or PR is relevant to the FHIR service in the Azure Healthcare APIs Bug Bug bug bug. Schema Version unchanged

Projects

None yet

Milestone

2Wk13

Development

Successfully merging this pull request may close these issues.

3 participants

@feordin @rbans96