Info-level logging when a remote webhook filter is empty

Description

SGW is not picking up the webhook if it's defined as filter:

The relevant part of the config:

 

"bucket":"data-bucket", "event_handlers": { "document_changed":[ {"handler": "webhook", "filter":"http://{IP}:5007/webhookFilter",   "url": "http://{IP}:5007"}]

 

The webhook filter:

@app.route("/webhookFilter") def webhook_filter(): return 'function(doc) { return false }'

The runtime config:

 

'event_handlers': {'document_changed': [{'handler': 'webhook', 'url': 'http://{IP}:5007'}] }

I have attached the logs. Presumably the filter is picked up because there would be an error in the logs.

 

SGW: 3.1.0-530

 

Attachments

2
  • 22 Feb 2023, 02:52 PM
  • 22 Feb 2023, 02:52 PM

Activity

Show:

CB robot March 1, 2023 at 3:00 PM

Build sync_gateway-3.1.0-543 contains sync_gateway commit 1bc073b with commit message:
: Add logging for absence of filter in webhook config (#6121)

Gilad Kalchheim February 24, 2023 at 11:11 AM

The problem was with missing dependencies in the test framework.   said we could add an error in the logs to show that the filter hasn't been retrieved, so i'm keeping is open.

Ben Brooks February 22, 2023 at 6:24 PM

Will attempt a repro next week after collections work is done if still unresolved

Gilad Kalchheim February 22, 2023 at 3:04 PM

I want to have another look, they could be a test issue.

Fixed
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Story Points

Sprint

Fix versions

Affects versions

Priority

Instabug

Open Instabug

PagerDuty

Sentry

Zendesk Support

Created February 22, 2023 at 2:53 PM
Updated April 13, 2023 at 11:25 PM
Resolved March 1, 2023 at 2:27 PM
Instabug