cockroachdb / cockroachdb/sequelize-cockroachdb

Telemetry creates error log when using `Sequelize` without active connections

Open
#155 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
55
Forks
18
PR merge metrics
No merged PRs in 30d

Description

Your `afterInit` hook assumes that the instance is actually connected, this may not be the case (especially in unit tests), I kindly request you add some code that checks if a connections exists before attempting to send telemetry.

Until then we are going to disable it via `cockroachdbTelemetryDisabled`.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.