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

OpenTelemetry: Document what information is collected and allow opt-out #2129

Closed
mtotschnig opened this issue Jun 24, 2024 · 2 comments
Closed
Assignees

Comments

@mtotschnig
Copy link

Is your feature request related to a problem? Please describe.
Users of my app want to know why it depends on OpenTelemetry: mtotschnig/MyExpenses#1523. I would love to be transparent about what information is collected by MSAL and ideally offer the option to opt-out.

Describe the solution you'd like

  • Documentation of how OpenTelemetry is used by this library.
  • An option that can be set in the config file or via a parameter that would allow to prevent any logging via OpenTelemetry.
@mtotschnig mtotschnig changed the title OpenTelemetry: Document what information is collected and allow opt-ou OpenTelemetry: Document what information is collected and allow opt-out Jun 24, 2024
@shahzaibj
Copy link
Contributor

shahzaibj commented Jul 9, 2024

Hi @mtotschnig Currently MSAL does NOT collect any telemetry data. The open telemetry has two parts: API (instrumentation layer) and SDK (export layer). MSAL currently is only using the API to instrument the code. The SDK part is never used in MSAL. Without the SDK part the data is never exported and all instrumentation from the API essentially becomes NO-OP without an SDK integrated.

A customer application can potentially take a dependency on SDK and register their own exporter and in that case the data may be collected and transported to customer's own backend/storage and in that case they will own the data. This is the only way the data can be collected from MSAL today and if a customer has configured this then we provide no support for any issues arising in this layer because this would be their own code/solution.

As things stand today, MSAL doesn't export any telemetry data by default and there is no advertised support for any 3p integration for data export.

I hope that helps.

@mtotschnig
Copy link
Author

Thanks @shahzaibj !

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