Notification merge requestshttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests2023-05-09T19:26:28Zhttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/385M18 service and CICD updates2023-05-09T19:26:28ZLong ChengM18 service and CICD updatesAWS code changes only
## Type of change
- [ ] Bug Fix
- [ ] Feature
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
## Does this introduce a change in the core logic?
- [YES/NO]
## Does this introduce ...AWS code changes only
## Type of change
- [ ] Bug Fix
- [ ] Feature
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
## Does this introduce a change in the core logic?
- [YES/NO]
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Azure
- [ ] Google Cloud
- [ ] IBM
## Does this introduce a breaking change?
- [YES/NO]
## What is the current behavior?
## What is the new/expected behavior?
## Have you added/updated Unit Tests and Integration Tests?
## Any other useful informationM18 - Release 0.21Long ChengLong Chenghttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/384fix global exception API response2023-05-04T17:32:21ZNeelesh Thakurfix global exception API responseAdded the missing comma in generating Json using String format
## Type of change
- [X] Bug Fix
- [ ] FeatureAdded the missing comma in generating Json using String format
## Type of change
- [X] Bug Fix
- [ ] FeatureM18 - Release 0.21https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/383Updated readme2023-08-18T15:23:59ZDmitrii Novikov (EPAM)Updated readmeUpdated pubsub config descriptionUpdated pubsub config descriptionM18 - Release 0.21Dmitrii Novikov (EPAM)Dmitrii Novikov (EPAM)https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/382Set FailOnError to false2023-05-03T19:42:48ZYash DholakiaSet FailOnError to false## Type of change
- [X] Bug Fix
- [ ] Feature
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
## Does this introduce a change in the core logic?
- [NO]
## Does this introduce a change in the cloud provi...## Type of change
- [X] Bug Fix
- [ ] Feature
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
## Does this introduce a change in the core logic?
- [NO]
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [X] AWS
- [ ] Azure
- [ ] Google Cloud
- [ ] IBM
## Does this introduce a breaking change?
- [NO]
## What is the current behavior?
Dependency-check-maven fails when nvd.nist is down and eventually build fails.
## What is the new/expected behavior?
Build will not fail when Dependency-check-maven fails
## Have you added/updated Unit Tests and Integration Tests?
## Any other useful informationM18 - Release 0.21Yash DholakiaYash Dholakiahttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/381Modify the version for springdoc2023-04-26T19:11:33ZVaibhavi KamaniModify the version for springdocModify the version for springdocModify the version for springdocVaibhavi KamaniVaibhavi Kamanihttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/380comment out notification-gc module2023-04-25T22:35:24ZOsborn Chencomment out notification-gc module## Type of change
- [ ] Bug Fix
- [ ] Feature
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
https://dev.azure.com/OpenEnergyPlatform/Open%20Energy%20Platform/_workitems/edit/23714
## Does this introduc...## Type of change
- [ ] Bug Fix
- [ ] Feature
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
https://dev.azure.com/OpenEnergyPlatform/Open%20Energy%20Platform/_workitems/edit/23714
## Does this introduce a change in the core logic?
- [YES/NO]
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Azure
- [ ] Google Cloud
- [ ] IBM
## Does this introduce a breaking change?
- [YES/NO]
## What is the current behavior?
https://dev.azure.com/OpenEnergyPlatform/Open%20Energy%20Platform/_workitems/edit/23714
## What is the new/expected behavior?
## Have you added/updated Unit Tests and Integration Tests?
## Any other useful informationOsborn ChenOsborn Chenhttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/379GONGR-6885 Reduce requests according to VPA2023-08-18T15:24:00ZAndriy Halka [EPAM / GCP]GONGR-6885 Reduce requests according to VPAM18 - Release 0.21https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/378Added a check for RabbitMQ Retry (GONRG-6705)2023-04-26T11:35:59ZRiabokon Stanislav(EPAM)[GCP]Added a check for RabbitMQ Retry (GONRG-6705)## Type of change
- [ ] Bug Fix
- [X] Feature
## Does this introduce a change in the core logic?
- [NO]
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Azure
- [X] Google Cloud...## Type of change
- [ ] Bug Fix
- [X] Feature
## Does this introduce a change in the core logic?
- [NO]
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Azure
- [X] Google Cloud
- [ ] IBM
## Does this introduce a breaking change?
- [NO]
Added a check for RabbitMQ Retry.M18 - Release 0.21Riabokon Stanislav(EPAM)[GCP]Riabokon Stanislav(EPAM)[GCP]https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/375Cherry pick changes from M14 to M16 master branch2023-04-17T22:44:07ZVaibhavi KamaniCherry pick changes from M14 to M16 master branchCherry pick changes from M14 to M16 master branchCherry pick changes from M14 to M16 master branchVaibhavi KamaniVaibhavi Kamanihttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/374remove logs that cause logs quota exhaustion2023-04-19T14:03:23ZYurii Kondakovremove logs that cause logs quota exhaustionHOTFIX. We saw log quota exhaustion in prod. The biggest culprit was notification service traces. We have identified 3 log entries that appear to be causing the bulk of the issues and these should be removed
1. Log message startswith "N...HOTFIX. We saw log quota exhaustion in prod. The biggest culprit was notification service traces. We have identified 3 log entries that appear to be causing the bulk of the issues and these should be removed
1. Log message startswith "Notification process started for message with id" - from **notification** service
2. Log message startswith "Start worker task : {messageId" - from core-lib-azure
3. Log message startswith "End worker task duration(ms)" from core-lib-azure
core-lib-azure logs(2,3) will be removed by MR
https://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/294M18 - Release 0.21Yurii KondakovYurii Kondakovhttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/373Updating NOTICE2023-04-13T14:00:05ZChad LeongUpdating NOTICEFIX NOTICEFIX NOTICEM17 - Release 0.20https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/372Merge branch 'gc-unhardcode-groupid-in-it' into 'master'2023-04-13T20:54:29ZRustam Lotsmanenko (EPAM)rustam_lotsmanenko@epam.comMerge branch 'gc-unhardcode-groupid-in-it' into 'master'Original merge https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/370
## Type of change
- [x] Bug Fix
- [ ] Feature
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
-
## Do...Original merge https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/370
## Type of change
- [x] Bug Fix
- [ ] Feature
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
-
## Does this introduce a change in the core logic?
- [NO]
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Azure
- [x] Google Cloud
- [ ] IBM
## Does this introduce a breaking change?
- [NO]
## What is the current behavior?
Group id is hardcoded in int tests.
## What is the new/expected behavior?
Group id can be configured via env vars.
## Have you added/updated Unit Tests and Integration Tests?
-
## Any other useful information
-M17 - Release 0.20Rustam Lotsmanenko (EPAM)rustam_lotsmanenko@epam.comRustam Lotsmanenko (EPAM)rustam_lotsmanenko@epam.comhttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/371GONRG-6935: Bring GROUP_ID variable to notification service2023-04-12T12:49:00ZMaksym Petrenko [EPAM / GCP]GONRG-6935: Bring GROUP_ID variable to notification service## Type of change
- [ ] Bug Fix
- [ ] Feature
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
## Does this introduce a change in the core logic?
- [YES/NO]
## Does this introduce a change in the cloud p...## Type of change
- [ ] Bug Fix
- [ ] Feature
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
## Does this introduce a change in the core logic?
- [YES/NO]
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Azure
- [ ] Google Cloud
- [ ] IBM
## Does this introduce a breaking change?
- [YES/NO]
## What is the current behavior?
## What is the new/expected behavior?
## Have you added/updated Unit Tests and Integration Tests?
## Any other useful informationMaksym Petrenko [EPAM / GCP]Maksym Petrenko [EPAM / GCP]https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/370unhardcode group id in it2023-04-13T16:44:35ZRustam Lotsmanenko (EPAM)rustam_lotsmanenko@epam.comunhardcode group id in it## Type of change
- [x] Bug Fix
- [ ] Feature
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
-
## Does this introduce a change in the core logic?
- [NO]
## Does this introduce a change in the cloud prov...## Type of change
- [x] Bug Fix
- [ ] Feature
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
-
## Does this introduce a change in the core logic?
- [NO]
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Azure
- [x] Google Cloud
- [ ] IBM
## Does this introduce a breaking change?
- [NO]
## What is the current behavior?
Group id is hardcoded in int tests.
## What is the new/expected behavior?
Group id can be configured via env vars.
## Have you added/updated Unit Tests and Integration Tests?
-
## Any other useful information
-M17 - Release 0.20Rustam Lotsmanenko (EPAM)rustam_lotsmanenko@epam.comRustam Lotsmanenko (EPAM)rustam_lotsmanenko@epam.comhttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/368OpenAPI 3.0 Documentation using springdoc2023-04-12T03:04:27ZThulasi Dass SubramanianOpenAPI 3.0 Documentation using springdoc
**Link to ADR(Architecture Decision Record)** : [Swagger using springdoc-openapi](https://community.opengroup.org/osdu/platform/system/home/-/issues/97)
## OpenAPI 3.0 related changes
* upgraded to latest **springdoc openapi** latest ...
**Link to ADR(Architecture Decision Record)** : [Swagger using springdoc-openapi](https://community.opengroup.org/osdu/platform/system/home/-/issues/97)
## OpenAPI 3.0 related changes
* upgraded to latest **springdoc openapi** latest version [1.6.14](https://mvnrepository.com/artifact/org.springdoc/springdoc-openapi-ui/1.6.14)
* used **[@OpenAPIDefinition](https://docs.swagger.io/swagger-core/v2.0.9/apidocs/io/swagger/v3/oas/annotations/OpenAPIDefinition.html)** annotation to populate OpenAPI Object fields *\[info, tags, servers, security\]*
* Documented the below API's with OpenAPI 3.0 **Annotations**
- PubSubEndpoint API
- Health Checks API
- Info API
* Added the standard HTTP Response(4xx, 5x\*\*\*\*x) for API Responses
* Custom Path for
* **Swagger UI**: https://host/context-path/swagger (will redirect to https://host/context-path/swagger-ui/index.html)
* **api-docs (JSON)** : https://host/context-path/api-docs
* **api-docs (YAML)** : https://host/context-path/api-docs.yaml
* Azure Swagger GLAB(for Reference)
* **Swagger UI**: https://osdu-glab.msft-osdu-test.org/api/notification/v1/swagger (will redirect to https://osdu-glab.msft-osdu-test.org/api/notification/v1/swagger-ui/index.html)
* **api-docs (JSON)** : https://osdu-glab.msft-osdu-test.org/api/notification/v1/api-docs
* **api-docs (YAML)** :https://osdu-glab.msft-osdu-test.org/api/notification/v1/api-docs.yaml
## Other Changes
- **Configurable** descriptions managed in [swagger.properties](https://community.opengroup.org/osdu/platform/system/partition/-/blob/az/td-oas/partition-core/src/main/resources/swagger.properties)
- Deleted 'HomeController'
- Marked the 'GlobalErrorController' as 'Hidden' for documentation
- Updated Readme for swagger related information
## References
- https://springdoc.org/faq.html#_can_i_use_spring_property_with_swagger_annotations
- https://springdoc.org/migrating-from-springfox.htmlM18 - Release 0.21Thulasi Dass SubramanianThulasi Dass Subramanianhttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/366AWS integration merge2023-04-07T12:44:55ZManish JangidAWS integration merge## Type of change
- [ ] Bug Fix
- [ ] Feature
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
## Does this introduce a change in the core logic?
- [YES/NO] No
## Does this introduce a change in the clou...## Type of change
- [ ] Bug Fix
- [ ] Feature
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
## Does this introduce a change in the core logic?
- [YES/NO] No
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [X ] AWS
- [ ] Azure
- [ ] Google Cloud
- [ ] IBM
## Does this introduce a breaking change?
- [YES/NO] no
## What is the current behavior?
## What is the new/expected behavior? AWS pom changes and aws devops changes
## Have you added/updated Unit Tests and Integration Tests? No
## Any other useful informationM18 - Release 0.21https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/364Close deleted subscriptions2023-03-30T18:40:33ZAlok JoshiClose deleted subscriptionsToday, the subscription manager in Notification service doesn't close/unregister deleted subscriptions for SB topics. In a scenario when a subscription is deleted, the registered client still attempts to pull messages from the subscripti...Today, the subscription manager in Notification service doesn't close/unregister deleted subscriptions for SB topics. In a scenario when a subscription is deleted, the registered client still attempts to pull messages from the subscription (which doesn't exist now).
This causes an explosion of error logs in the Trace table. The log message looks like this:
`Exception RECEIVE occurred in service bus message in exception phase The messaging entity '<service-bus-resource>:topic:recordstopic~15|<notification-id>' could not be found`
The number of such logs is very very large, they only go away when service is restarted.
This solution closes the connections for which the subscription doesn't exist in the cosmosDbM17 - Release 0.20Alok JoshiChad LeongAlok Joshihttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/363changes to fix s360 vulnerabilities2023-03-28T14:08:43ZAbhiram Bondadachanges to fix s360 vulnerabilities## Type of change
- [x] Bug Fix
- [ ] Feature
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
## Does this introduce a change in the core logic?
- NO
## Does this introduce a change in the cloud provide...## Type of change
- [x] Bug Fix
- [ ] Feature
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
## Does this introduce a change in the core logic?
- NO
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [x] Azure
- [ ] Google Cloud
- [ ] IBM
## Does this introduce a breaking change?
- NO
## What is the current behavior?
## What is the new/expected behavior?
## Have you added/updated Unit Tests and Integration Tests?
## Any other useful informationhttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/361Vulnerability fix for IBM2023-03-31T08:22:49ZDevdatta SantraVulnerability fix for IBMVulnerability fix for IBM for `jackson` lib
| CVE-2022-42004 | https://nvd.nist.gov/vuln/detail/CVE-2022-42004 |
|----------------|-------------------------------------------------|
| CVE-2022-42003 | https://nvd.nist.gov/vuln/detail/CV...Vulnerability fix for IBM for `jackson` lib
| CVE-2022-42004 | https://nvd.nist.gov/vuln/detail/CVE-2022-42004 |
|----------------|-------------------------------------------------|
| CVE-2022-42003 | https://nvd.nist.gov/vuln/detail/CVE-2022-42003 |M17 - Release 0.20Devdatta SantraDevdatta Santrahttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/359[GONRG-6649] Define global vars in notification2023-08-18T15:24:02ZOleksandr Kosse (EPAM)[GONRG-6649] Define global vars in notification## Type of change
- [ ] Bug Fix
- [ ] Feature
- [X] Refactoring
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
## Does this introduce a change in the core logic?
- [NO]
## Does this introduce a change ...## Type of change
- [ ] Bug Fix
- [ ] Feature
- [X] Refactoring
**Please provide link to gitlab issue or ADR(Architecture Decision Record)**
## Does this introduce a change in the core logic?
- [NO]
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Azure
- [X] Google Cloud
- [ ] IBM
## Does this introduce a breaking change?
- [NO]
## What is the current behavior?
## What is the new/expected behavior?
## Have you added/updated Unit Tests and Integration Tests?
## Any other useful informationM17 - Release 0.20