influxdata / influxdata/telegraf
Migrate from couchbase/go-couchbase to couchbase/gocb
- 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
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