Sign upLogin

horizoncd/horizon

View on GitHub
Star
pkg/eventhandler/wlgenerator/wlgenerator.go

Summary

Maintainability
C
1 day
Test Coverage

Showing 4 of 4 total issues

Method WebhookLogGenerator.Process has a Cognitive Complexity of 38 (exceeds 20 allowed). Consider refactoring.
Open

func (w *WebhookLogGenerator) Process(ctx context.Context, events []*models.Event,
    resume bool) error {
    var (
        webhookLogs        []*webhookmodels.WebhookLog
        conditionsToCreate = map[uint]map[uint]messageDependency{}
Severity: Minor
Found in pkg/eventhandler/wlgenerator/wlgenerator.go - About 3 hrs to fix

Method WebhookLogGenerator.Process has 88 lines of code (exceeds 50 allowed). Consider refactoring.
Open

func (w *WebhookLogGenerator) Process(ctx context.Context, events []*models.Event,
    resume bool) error {
    var (
        webhookLogs        []*webhookmodels.WebhookLog
        conditionsToCreate = map[uint]map[uint]messageDependency{}
Severity: Major
Found in pkg/eventhandler/wlgenerator/wlgenerator.go - About 2 hrs to fix

    Method WebhookLogGenerator.makeRequestBody has 73 lines of code (exceeds 50 allowed). Consider refactoring.
    Open

    func (w *WebhookLogGenerator) makeRequestBody(ctx context.Context, dep *messageDependency) (string, error) {
        message := MessageContent{
            EventID:   dep.event.ID,
            WebhookID: dep.webhook.ID,
            EventType: dep.event.EventType,
    Severity: Minor
    Found in pkg/eventhandler/wlgenerator/wlgenerator.go - About 1 hr to fix

      Method WebhookLogGenerator.makeRequestBody has 5 return statements (exceeds 4 allowed).
      Open

      func (w *WebhookLogGenerator) makeRequestBody(ctx context.Context, dep *messageDependency) (string, error) {
          message := MessageContent{
              EventID:   dep.event.ID,
              WebhookID: dep.webhook.ID,
              EventType: dep.event.EventType,
      Severity: Major
      Found in pkg/eventhandler/wlgenerator/wlgenerator.go - About 35 mins to fix
        Category
        Status