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

Extend telemetry.enabled description in config.sample.php -> Only enterprise users affected #41285

Closed
iasdeoupxe opened this issue Jul 8, 2024 · 2 comments

Comments

@iasdeoupxe
Copy link
Contributor

iasdeoupxe commented Jul 8, 2024

What?

Addition suggestion for:

I'm not good in providing texts so just raising this issue here instead of providing a PR.

Suggestion

The change would raise red alerts for me as a community user as "Telemetry" is enabled by default but looks like only enterprise users (with valid licenses) are affected -> owncloud/configreport#197 (comment)

Extend the text from #41283 to make this more clear, otherwise this will raise concerns by community users.

Possibly to be clarified and documented as well: Is this also only sending data if the configreport app is enabled or independent from that?

jnweiger added a commit that referenced this issue Jul 8, 2024
As motivated in #41285
Thanks for the hint.
I'd like to include a link to the documentation here, but that link is not yet available -> owncloud/docs-server#1350
@jnweiger
Copy link
Contributor

jnweiger commented Jul 8, 2024

Clarified the enterprise only featue in the text in config.sample.php - thanks for the hint.

Nothing can be sent, when the configreport app is disabled. E.g. the background job for sending telemetry is only present in occ background:queue:status while the app is enabled.

jnweiger added a commit that referenced this issue Jul 9, 2024
* docs: update telemetry.enabled in config.sample.php

As motivated in #41285
Thanks for the hint.
I'd like to include a link to the documentation here, but that link is not yet available -> owncloud/docs-server#1350

* docs: update config/config.sample.php
improve wording around telemetry.enabled

Co-authored-by: iasdeoupxe <[email protected]>

---------

Co-authored-by: iasdeoupxe <[email protected]>
@iasdeoupxe
Copy link
Contributor Author

#41286 is already merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants