influxdata / influxdata/telegraf

Migrate from couchbase/go-couchbase to couchbase/gocb

Open
#9,813 1 comment 0 reactions 0 assignees View on GitHub
area/couchbase help wanted size/l
Dominant language
Go
Stars
17.8k
Forks
5.8k
Avg merge
1d 20h
Merged PRs (30d)
161

Description

Couchbase provides two go client modules. couchbase/gocb is the current official module and couchbase/go-couchbase is an older unofficial module.

The telegraf couchbase input uses couchbase/go-couchbase. We have recently had to work around missing functionality (#9032) and insecure code (#9485) in it.

Telegraf should switch to gocb because it is the official supported module.

There is an old closed issue to switch to gocb (#2418). At the time of that issue the gocb module was new and didn't have functionality telegraf needed. We should revisit this now that time has past and gocb has matured.

Contributor guide

Open the contributing guide

Research direction

Start with Telegraf's Couchbase input and inspect its current use of couchbase/go-couchbase, then read issues #9032 and #9485 for functionality and security constraints. Revisit issue #2418 and verify that the input can use couchbase/gocb while retaining the required behavior and resolving the cited limitations.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
database
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.