OS Core Lib Azure merge requestshttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests2023-08-18T12:46:18Zhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/36adds ability use own ElasticCredentialsCache impl2023-08-18T12:46:18ZAliaksei Darafeyeuadds ability use own ElasticCredentialsCache implM1 - Release 0.1Neelesh ThakurAlok JoshiDuvelis CaraoNeelesh Thakurhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/335Java 17 migration2024-03-21T05:54:33ZVidyaDharani LokamJava 17 migration# Description:
Migration from Java 8 to Java 17.
# How to test:
Via Unit and Integration tests.
# Changes include:
* [x] Breaking change (a change that is not backward-compatible and/or changes current functionality).
# Changes in:...# Description:
Migration from Java 8 to Java 17.
# How to test:
Via Unit and Integration tests.
# Changes include:
* [x] Breaking change (a change that is not backward-compatible and/or changes current functionality).
# Changes in:
* [x] Common code
# Dev Checklist:
* [ ] Added Unit Tests, wherever applicable.
* [x] Updated the Readme, if applicable.
* [x] Existing Tests pass
* [x] Verified functionality locally
* [ ] Self Reviewed my code for formatting and complex business logic.
# Other comments:
* Upgraded 'lombok' & 'jacoco' plugin version to generate code-coverage report
* Changed TestRunner 'PowerMockRunner' to 'MockitoJUnitRunner'
* Removed 'powermock-api-mockito2' & 'powermock-module-junit4' dependencies
* Updated existing testcases to use junit-5M23 - Release 0.26VidyaDharani LokamVidyaDharani Lokamhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/248increase version of os-core-common to 0.18.0-rc22023-08-18T12:42:40ZYauheni Lesnikauincrease version of os-core-common to 0.18.0-rc2Increase version of os-core-common with to 0.18.0-rc2 fix of whitesource security vulnerabilities.
Merged MR of os-core-common: https://community.opengroup.org/osdu/platform/system/lib/core/os-core-common/-/merge_requests/179
Fix white...Increase version of os-core-common with to 0.18.0-rc2 fix of whitesource security vulnerabilities.
Merged MR of os-core-common: https://community.opengroup.org/osdu/platform/system/lib/core/os-core-common/-/merge_requests/179
Fix whitesource high severity security vulnerabilities for next dependencies:
snakeyaml-1.26.jar - CVE-2022-25857
jackson-databind-2.13.2 - CVE-2022-42004M15 - Release 0.18Yauheni LesnikauYauheni Lesnikauhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/222Updated spring boot version2022-09-15T09:46:06ZDmitrii Novikov (EPAM)Updated spring boot versionhttps://community.opengroup.org/osdu/platform/system/indexer-service/-/issues/56
- Updated spring-boot version 2.4.12 -> 2.7.2, lombok version 1.18.16 -> 1.18.24
- Fixed issue with deprecated actuator classeshttps://community.opengroup.org/osdu/platform/system/indexer-service/-/issues/56
- Updated spring-boot version 2.4.12 -> 2.7.2, lombok version 1.18.16 -> 1.18.24
- Fixed issue with deprecated actuator classesM13 - Release 0.16https://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/46Explicit method to generate a PreSigned URL along with SAS Token2023-08-18T12:46:04ZKrishna Nikhil VedurumudiExplicit method to generate a PreSigned URL along with SAS Token## All Submissions:
-------------------------------------
* [YES] I have added an explanation of what changes in this merge do and why we should include it?
* [YES] I have updated the documentation accordingly.
* [NO] I have added tests ...## All Submissions:
-------------------------------------
* [YES] I have added an explanation of what changes in this merge do and why we should include it?
* [YES] I have updated the documentation accordingly.
* [NO] I have added tests to cover my changes.
* [YES] All new and existing tests passed.
* [YES] My code follows the code style of this project.
* [YES] I ran lint checks locally prior to submission.
## What is the issue or story related to the change?
-------------------------------------
The idea is to have common code even for the SignedURL generation. Using BlockBlobClient is safer approach as it takes care of encoding related aspects and in every service where the Signed URLs are required.
Handling construction of URLs in service level could very well lead to Double Encoding issues.
High level design:
Issue: <!-- Link any __GitLab__ workitem(s) to this pull request. -->
<!-- Please add implementation details of current set of changes and how the code changes are
doing what they are expected to do. Are there any complex loops or designated code blocks that
should be elaborated? Is there some contextual knowledge that the reviewer should be aware of? -->
Change details:
## Test coverage:
------------------
<!-- Mention unit test coverage of changes. -->
## Does this introduce a breaking change?
-------------------------------------
- [NO]
<!-- If this introduces a breaking change, please describe the impact and migration path for existing applications below. -->
## Pending items
----------------
<!-- Are there changes that you'll introduce in upcoming MRs and hence did not add in this one? Next steps of your
feature can also be mentioned here. -->
## Reviewer request
-------------------
- Please provide an ETA when you plan to review this MR. Write a comment to decline or provide an ETA.
- Block the MR if you feel there is less testing or no details in the MR
- Please cover the following aspects in the MR
-- Coding design: _\<Reviewer1>_
-- Backward Compatibility: _\<Reviewer2>_
-- Feature Logic: _\<Logic design\>_
-- _\<Any other context mention here>_
OR
-- _\<Component 1>_: _\<Reviewer1>_
-- _\<CosmosDB>_: _\<Reviewer2>_
-- _\<ServiceBus>_ _\<Reviewer3>_
-- _\<Mention any other component and owner>_
## Other information
-------------------------------------
<!-- Any other information that is important to this MR such as screenshots of how the component looks before and after the change. -->M1 - Release 0.1Kishore BattulaKishore Battulahttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/40uses defaultAzureCredential instead of key for cosmosClient2020-11-05T15:09:48ZAliaksei Darafeyeuuses defaultAzureCredential instead of key for cosmosClientNitin-slbNeelesh ThakurSherman YangAlok JoshiDuvelis CaraoNitin-slbhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/30adds queryItemsWithContinuationToken to CosmosStore2020-10-02T15:11:03ZAliaksei Darafeyeuadds queryItemsWithContinuationToken to CosmosStoreNitin-slbNeelesh ThakurPavel BachylaAlok JoshiDuvelis CaraoNitin-slbhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/19Updating the MR template2023-08-18T12:46:44ZAman VermaUpdating the MR template## All Submissions:
-------------------------------------
* [YES/NO] I have added an explanation of what changes in this merge do and why we should include it?
YES
* [YES/NO] I have updated the documentation accordingly.
YES
* [YES/NO/...## All Submissions:
-------------------------------------
* [YES/NO] I have added an explanation of what changes in this merge do and why we should include it?
YES
* [YES/NO] I have updated the documentation accordingly.
YES
* [YES/NO/NA] I have added tests to cover my changes.
NA
* [YES/NO/NA] All new and existing tests passed.
NA
* [YES/NO/NA] My code follows the code style of this project.
NA
* [YES/NO/NA] I ran lint checks locally prior to submission.
NA
## What is the issue or story related to the change?
-------------------------------------
<!-- Please describe the current behavior that you are modifying, 'or' link to a relevant issue. -->
Description:
updating the MR template to add new line items.
Issue: <!-- Link any __GitLab__ workitem(s) to this pull request. -->
## Does this introduce a breaking change?
-------------------------------------
- [YES/NO]
NO
<!-- If this introduces a breaking change, please describe the impact and migration path for existing applications below. -->
## Other information
-------------------------------------
<!-- Any other information that is important to this PR such as screenshots of how the component looks before and after the change. -->
@kibattul, @danielscholl FYIM1 - Release 0.1Aman VermaAman Vermahttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/5Added transaction logging2020-07-24T14:56:09ZKishore BattulaAdded transaction logging## All Submissions:
-------------------------------------
* [YES] I have added an explanation of what changes in this merge do and why we should include it?
* [YES] I have updated the documentation accordingly.
* [YES] I have added t...## All Submissions:
-------------------------------------
* [YES] I have added an explanation of what changes in this merge do and why we should include it?
* [YES] I have updated the documentation accordingly.
* [YES] I have added tests to cover my changes.
* [YES] All new and existing tests passed.
* [YES] My code follows the code style of this project.
* [YES/NO/NA] I ran lint checks locally prior to submission.
## What is the issue or story related to the change?
-------------------------------------
Adding a slf4jLogger implementation for ILogger interface. Added logs to record start and end of a request
Description:
Issue: <!-- Link any __GitLab__ workitem(s) to this pull request. -->
## Does this introduce a breaking change?
-------------------------------------
- [NO]
<!-- If this introduces a breaking change, please describe the impact and migration path for existing applications below. -->
## Other information
-------------------------------------
<!-- Any other information that is important to this PR such as screenshots of how the component looks before and after the change. -->
Daniel SchollHema Vishnu Pola [Microsoft]Daniel Schollhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/340Added oid validation2024-03-20T11:36:24ZDeepa KumariAdded oid validationOID validation addition to address the issue: https://community.opengroup.org/osdu/platform/security-and-compliance/entitlements/-/issues/166OID validation addition to address the issue: https://community.opengroup.org/osdu/platform/security-and-compliance/entitlements/-/issues/166M23 - Release 0.26Deepa KumariDeepa Kumarihttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/339[MSOSDU-36231]Added OID Validation utilities and tests2024-03-12T11:59:35ZDeepa Kumari[MSOSDU-36231]Added OID Validation utilities and testsAdded interaction with Graph API to validate OID. Related issue: https://community.opengroup.org/osdu/platform/security-and-compliance/entitlements/-/issues/166Added interaction with Graph API to validate OID. Related issue: https://community.opengroup.org/osdu/platform/security-and-compliance/entitlements/-/issues/166M23 - Release 0.26Deepa KumariDeepa Kumarihttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/338Move Spring upgrade to a different module2024-03-11T07:07:18ZVidyaDharani LokamMove Spring upgrade to a different module# Description:
- Added core-lib-azure-spring6 module compatible with spring 6.
- Migrated to jakarta from javax in spring6 module.
# Changes include:
* [x] Refactor (a non-breaking change that improves code maintainability).
* [ ] Bug...# Description:
- Added core-lib-azure-spring6 module compatible with spring 6.
- Migrated to jakarta from javax in spring6 module.
# Changes include:
* [x] Refactor (a non-breaking change that improves code maintainability).
* [ ] Bugfix (a non-breaking change that solves an issue).
* [ ] New feature (a non-breaking change that adds functionality).
* [ ] Breaking change (a change that is not backward-compatible and/or changes current functionality).M23 - Release 0.26VidyaDharani LokamVidyaDharani Lokamhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/337update documentation in readme2024-03-19T17:14:21ZVidyaDharani Lokamupdate documentation in readme- Added the Jacoco code coverage report in the README.
- Updated the prerequisites with links.- Added the Jacoco code coverage report in the README.
- Updated the prerequisites with links.M23 - Release 0.26VidyaDharani LokamVidyaDharani Lokamhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/336Draft: update spring and spring-boot2024-03-11T07:07:29ZVidyaDharani LokamDraft: update spring and spring-bootVidyaDharani LokamVidyaDharani Lokamhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/334Draft: Java 17 migration2024-02-23T08:24:48ZVidyaDharani LokamDraft: Java 17 migration## All Submissions:
-------------------------------------
* [YES/NO] I have added an explanation of what changes in this merge do and why we should include it?
* [YES/NO] I have updated the documentation accordingly.
* [YES/NO/NA] I have...## All Submissions:
-------------------------------------
* [YES/NO] I have added an explanation of what changes in this merge do and why we should include it?
* [YES/NO] I have updated the documentation accordingly.
* [YES/NO/NA] I have added tests to cover my changes.
* [YES/NO/NA] All new and existing tests passed.
* [YES/NO/NA] My code follows the code style of this project.
* [YES/NO/NA] I ran lint checks locally prior to submission.
## What is the issue or story related to the change?
-------------------------------------
<!-- Please describe the current behavior that you are modifying, 'or' link to a relevant issue.
Feel free to add references to any design documents you might have shared with the team or any
related MR that you are building on top of. -->
High level design:
Issue: <!-- Link any __GitLab__ workitem(s) to this pull request. -->
<!-- Please add implementation details of current set of changes and how the code changes are
doing what they are expected to do. Are there any complex loops or designated code blocks that
should be elaborated? Is there some contextual knowledge that the reviewer should be aware of? -->
Change details:
## Test coverage:
------------------
<!-- Mention unit test coverage of changes. -->
## Does this introduce a breaking change?
-------------------------------------
- [YES/NO]
<!-- If this introduces a breaking change, please describe the impact and migration path for existing applications below. -->
## Pending items
----------------
<!-- Are there changes that you'll introduce in upcoming MRs and hence did not add in this one? Next steps of your
feature can also be mentioned here. -->
## Reviewer request
-------------------
- Please provide an ETA when you plan to review this MR. Write a comment to decline or provide an ETA.
- Block the MR if you feel there is less testing or no details in the MR
- Please cover the following aspects in the MR
-- Coding design: _\<Reviewer1>_
-- Backward Compatibility: _\<Reviewer2>_
-- Feature Logic: _\<Logic design\>_
-- _\<Any other context mention here>_
OR
-- _\<Component 1>_: _\<Reviewer1>_
-- _\<CosmosDB>_: _\<Reviewer2>_
-- _\<ServiceBus>_ _\<Reviewer3>_
-- _\<Mention any other component and owner>_
## Other information
-------------------------------------
<!-- Any other information that is important to this MR such as screenshots of how the component looks before and after the change. -->VidyaDharani LokamVidyaDharani Lokamhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/333Create Release candidate 3 from M22 milestone2024-02-16T07:47:41ZAnkur RawatCreate Release candidate 3 from M22 milestoneUpdated version of the libraryUpdated version of the libraryAnkur RawatAnkur Rawathttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/332429 exception handling for bulk update operation in storage2024-02-15T07:02:26ZAnkur Rawat429 exception handling for bulk update operation in storageCurrently, bulk insertion in cosmos DB functionality returns 500 in case of 429 error form cosmos DB. Updated the functionality to send 429 received from Cosmos DBCurrently, bulk insertion in cosmos DB functionality returns 500 in case of 429 error form cosmos DB. Updated the functionality to send 429 received from Cosmos DBAnkur RawatAnkur Rawathttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/331CosmosDB 429 Exception handling in Azure2024-02-13T13:53:38ZAnkur RawatCosmosDB 429 Exception handling in AzureCurrently, bulk insertion in cosmos DB functionality returns 500 in case of 429 error form cosmos DB. Updated the functionality to send 429 received from Cosmos DB.Currently, bulk insertion in cosmos DB functionality returns 500 in case of 429 error form cosmos DB. Updated the functionality to send 429 received from Cosmos DB.Ankur RawatAnkur Rawathttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/330CosmosDB 429 Exception handling in Azure2024-02-13T13:46:58ZAnkur RawatCosmosDB 429 Exception handling in AzureCurrently, bulk insertion in cosmos DB functionality returns 500 in case of 429 error form cosmos DB. Updated the functionality to send 429 received from Cosmos DB.Currently, bulk insertion in cosmos DB functionality returns 500 in case of 429 error form cosmos DB. Updated the functionality to send 429 received from Cosmos DB.Ankur RawatAnkur Rawathttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/329CosmosDB 429 Exception handling in Azure2024-02-13T10:37:56ZAnkur RawatCosmosDB 429 Exception handling in AzureCurrently, bulk insertion in cosmos DB functionality returns 500 in case of 429 error form cosmos DB. Updated the functionality to send 429 received from Cosmos DB.Currently, bulk insertion in cosmos DB functionality returns 500 in case of 429 error form cosmos DB. Updated the functionality to send 429 received from Cosmos DB.Ankur RawatAnkur Rawat