search

microsoft / cpp_client_telemetry

1DS C++ SDK
Apache License 2.0
85 stars 48 forks source link

Processing PowerInfo broadcast async #1229

Open ajadusum opened 4 months ago

ajadusum commented 4 months ago

Issue: PowerInfoReceiver in HttpClient is processing broadcast on main thread. Android guideline is to avoid working on main thread on broadcast.

Fix: HttpClient already has a executor, processing the broadcast message async on the executor.

lalitb commented 4 months ago

@ajadusum The android build is failing. Can you check if this is related to your changes ?

ajadusum commented 4 months ago

@ajadusum The android build is failing. Can you check if this is related to your changes ?

Yes, a test is failing. I will fix the mocks tomorrow.