Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Telemetry] Opt-in/out notifications from server should not crash the Kibana request #75637

Closed
afharo opened this issue Aug 21, 2020 · 2 comments · Fixed by #75641
Closed

[Telemetry] Opt-in/out notifications from server should not crash the Kibana request #75637

afharo opened this issue Aug 21, 2020 · 2 comments · Fixed by #75641
Assignees
Labels
Feature:Telemetry Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v7.9.1

Comments

@afharo
Copy link
Member

afharo commented Aug 21, 2020

Kibana version: 7.9

Describe the bug:

When calling /api/telemetry/v2/optIn, Kibana notifies the remote telemetry service about the user's final decision. If the user opted-out, this will be the very last notification we send to the remote telemetry service.

If config.sendUsageFrom === 'server' but the server does not have access to the telemetry.elastic.co domain, the API call returns 500, not allowing the user to dismiss the telemetry banner.

Expected behavior:

The user's decision should be stored and the server should handle nicely if it can't reach the telemetry service.

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-telemetry (Team:KibanaTelemetry)

@afharo afharo self-assigned this Aug 21, 2020
@lukeelmers lukeelmers added the Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc label Oct 1, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core (Team:Core)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Telemetry Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v7.9.1
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants