Notification merge requestshttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests2021-08-30T21:23:01Zhttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/105Remove explicit lowercase action of notificationId when extracting from input...2021-08-30T21:23:01ZAlok JoshiRemove explicit lowercase action of notificationId when extracting from input BodyNotification service queries for a Subscription with `notificationId` during the /push-handlers/records-changes workflow. This call from Notification service to the Register service fails if the notificationId contains upper case charact...Notification service queries for a Subscription with `notificationId` during the /push-handlers/records-changes workflow. This call from Notification service to the Register service fails if the notificationId contains upper case characters. The reason being, we are lower casing the notificationId before querying Register service, which will return an empty list because its not an exact match.
This issue was observed with the test subscription with notificationId `testingAcknowledgement` in our environment.
Unless there is a specific reason we are lowercasing the notificationId, we can remove it.Alok JoshiAlok Joshihttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/106Develop helm config-map and helm deployments for Notification service [GONRG-...2021-09-29T09:40:40ZArmen Gasparyan (EPAM)Develop helm config-map and helm deployments for Notification service [GONRG-3036]- Develop configmap helms for notification service
- Develop deployment helms for notifocation service- Develop configmap helms for notification service
- Develop deployment helms for notifocation serviceOleksandr Kosse (EPAM)Oleksandr Kosse (EPAM)https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/111Ignore IT Conditionally2022-03-16T17:07:50ZNikhil Singh[MicroSoft]Ignore IT ConditionallyThis MR runs the integration test conditionally for event grid or service busThis MR runs the integration test conditionally for event grid or service busNikhil Singh[MicroSoft]Nikhil Singh[MicroSoft]https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/113Draft: ignorinng tests2022-03-16T17:07:48ZKomal MakkarDraft: ignorinng testshttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/115IT Update2022-03-16T17:07:49ZNikhil Singh[MicroSoft]IT UpdateNikhil Singh[MicroSoft]Nikhil Singh[MicroSoft]https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/120Add hpa config & memory limit setup with feature flag2022-07-12T15:36:30ZMina OtgonboldAdd hpa config & memory limit setup with feature flagAdd hpa config & memory limit setup with feature flagAdd hpa config & memory limit setup with feature flagM9 - Release 0.12https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/123BugFix: Added the missing comma in gemerating Json using String format2023-05-03T15:16:03ZHouari ZegaiBugFix: Added the missing comma in gemerating Json using String format## 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?
- [YES]
## 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?
- [YES]
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Azure
- [ ] GCP
- [ ] IBM
## Does this introduce a breaking change?
- [NO]
## What is the current behavior?
## What is the new/expected behavior?
Added the missing comma in generating Json using String format
## Have you added/updated Unit Tests and Integration Tests?
## Any other useful informationhttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/124Custom error for bad push url2022-04-18T13:55:56ZAlok JoshiCustom error for bad push urlAs of today, Notification service does not log which client (subscription) is corrupted/unable to process requests that are sent to the pushUrl as part of the notifySubscriber method. To narrow down on bad clients, we can log more detail...As of today, Notification service does not log which client (subscription) is corrupted/unable to process requests that are sent to the pushUrl as part of the notifySubscriber method. To narrow down on bad clients, we can log more details about the subscription. Also, we can return a custom Http status (424 FAILED_DEPENDENCY) to separate out all such errors in monitoring.M9 - Release 0.12Alok JoshiAlok Joshihttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/127update core common -Notification2021-10-22T13:29:34ZNikhil Singh[MicroSoft]update core common -Notification**Introduction**
This MR upgrades the os core-common version for azure provider to verify the recent security fixes made in the below mentioned MR are working properly.
**Issue Link**
https://community.opengroup.org/osdu/platform/syst...**Introduction**
This MR upgrades the os core-common version for azure provider to verify the recent security fixes made in the below mentioned MR are working properly.
**Issue Link**
https://community.opengroup.org/osdu/platform/system/lib/core/os-core-common/-/merge_requests/119Nikhil Singh[MicroSoft]Nikhil Singh[MicroSoft]https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/129Update to use latest core-common2021-10-14T20:55:58ZLarissa PereiraUpdate to use latest core-commonPlease refer to this issue [#90](https://community.opengroup.org/osdu/platform/system/home/-/issues/90)Please refer to this issue [#90](https://community.opengroup.org/osdu/platform/system/home/-/issues/90)https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/134Upgrading OSDU Dependencies2022-01-25T17:26:43ZDavid Diederichd.diederich@opengroup.orgUpgrading OSDU DependenciesDavid Diederichd.diederich@opengroup.orgDavid Diederichd.diederich@opengroup.orghttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/135Use Fixed Core Lib2022-04-18T13:55:57ZNikhil Singh[MicroSoft]Use Fixed Core LibNikhil Singh[MicroSoft]Nikhil Singh[MicroSoft]https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/143fix whitesource vulnerabilities2021-12-15T16:39:30ZYauheni Lesnikaufix whitesource vulnerabilitiesYauheni LesnikauYauheni Lesnikauhttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/145Added cache for Subscription not found2022-07-12T15:36:28ZSanjeev-SLBAdded cache for Subscription not foundAdded cache for Subscription not foundAdded cache for Subscription not foundSanjeev-SLBSanjeev-SLBhttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/146added istio dns host2021-12-13T04:31:34ZAnkur Rawatadded istio dns hostFix for istioDnsHost as empty string is not allowed.Fix for istioDnsHost as empty string is not allowed.Ankur RawatAnkur Rawathttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/151version update of core lib and core common2022-07-12T15:36:28ZAkanksha Prasadversion update of core lib and core commonUpdating core lib azure and core common versionsUpdating core lib azure and core common versionsAkanksha PrasadAkanksha Prasadhttps://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/155Update core lib2021-12-16T16:49:04ZNikhil Singh[MicroSoft]Update core libNikhil Singh[MicroSoft]Nikhil Singh[MicroSoft]https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/156update pom2022-07-12T15:36:33ZNikhil Singh[MicroSoft]update pomNikhil Singh[MicroSoft]Nikhil Singh[MicroSoft]https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/163Testing MR-Update .gitlab-ci.yml2022-01-03T09:52:30ZNikhil Singh[MicroSoft]Testing MR-Update .gitlab-ci.ymlNikhil Singh[MicroSoft]Nikhil Singh[MicroSoft]https://community.opengroup.org/osdu/platform/system/notification/-/merge_requests/166added logs to process time,topic, status and status code for each notification.2022-08-05T06:23:44ZSmitha Manjunathadded logs to process time,topic, status and status code for each notification.Notification V2 should provide detailed logs to capture all the failed/success messages including duration.Notification V2 should provide detailed logs to capture all the failed/success messages including duration.Smitha ManjunathSmitha Manjunath