Skip to content

add: notifications - add register origins method#287

Merged
mgoetzegb merged 1 commit intomainfrom
VTI-785-register-origins
Feb 16, 2026
Merged

add: notifications - add register origins method#287
mgoetzegb merged 1 commit intomainfrom
VTI-785-register-origins

Conversation

@mgoetzegb
Copy link
Copy Markdown
Member

What

add: notifications - add register origins method

Why

Allow conveniently calling the newly added endpoint (greenbone/opensight-notification-service#185).

Each service using the notification service is supposed to register its used (classes of) origins once on service startup.

References

VTI-785

@mgoetzegb mgoetzegb requested review from a team as code owners February 12, 2026 12:43
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 12, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 8823d66.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@github-actions
Copy link
Copy Markdown

Conventional Commits Report

Type Number
Added 1

🚀 Conventional commits found.

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 12, 2026

Codecov Report

❌ Patch coverage is 0% with 25 lines in your changes missing coverage. Please review.
✅ Project coverage is 57.34%. Comparing base (32c62c5) to head (8823d66).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
pkg/notifications/notification.go 0.00% 25 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #287      +/-   ##
==========================================
- Coverage   57.71%   57.34%   -0.38%     
==========================================
  Files          69       69              
  Lines        3862     3887      +25     
==========================================
  Hits         2229     2229              
- Misses       1445     1470      +25     
  Partials      188      188              
Flag Coverage Δ
opensearch-tests 95.66% <ø> (ø)
postgres-tests 91.96% <ø> (ø)
unit-tests 51.91% <0.00%> (-0.39%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mgoetzegb mgoetzegb force-pushed the VTI-785-register-origins branch from c600218 to 8823d66 Compare February 16, 2026 16:07
@mgoetzegb mgoetzegb enabled auto-merge (squash) February 16, 2026 16:07
@mgoetzegb mgoetzegb added the minor release Set label to create a minor release label Feb 16, 2026
@mgoetzegb mgoetzegb merged commit eaceb19 into main Feb 16, 2026
13 checks passed
@mgoetzegb mgoetzegb deleted the VTI-785-register-origins branch February 16, 2026 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor release Set label to create a minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants