search

Azure / diagnostics-eventflow

Microsoft Diagnostics EventFlow
MIT License
305 stars 97 forks source link

Shall we remove unused `Microsoft.Diagnostics.EventFlow.TableStorage`? #396

Closed xiaomi7732 closed 2 years ago

xiaomi7732 commented 3 years ago

Hi everyone,

After touch base with @karolz-ms, we reached an agreement to remove Microsoft.Diagnostics.EventFlow.TableStorage due to lack of usability, and in the sake of having a reduced attacking surface for vulnerability.

But we want to hear everyone's opinion, specially, ones that vote against the decision, just in case there's some hidden scenarios that we should've realized. Please comment below if you think there's a scenario for you and the project is worth keeping.

Details The package was developed at the beginning of the project but never since released a NuGet package for it. And there's no inquiries about it either.

karolz-ms commented 2 years ago

Resolved via https://github.com/Azure/diagnostics-eventflow/pull/398 -- we did remove it