clinthuffman / Clue

Triggered Windows performance data collection tool.
MIT License
98 stars 22 forks source link

Does EmailReportTo work as advertised in 1.1.6? #2

Open Borgquite opened 2 years ago

Borgquite commented 2 years ago

Love the tool & just using it for the first time here. I have configured EmailReportTo and entered an email address - but I can't see where I can configure (say) an SMTP server, not can I see anywhere in the code where the EmailReportTo value is processed.

Is this feature working?

clinthuffman commented 2 years ago

Sorry, I never got that feature working fully. For now, all it does is put the email address into the zip file that gets collected, so that whomever or whatever analyzes the data would know where to send the response to.

Thank you,

From: Chris @.> Sent: Wednesday, December 1, 2021 8:16 AM To: @.> Cc: @.***> Subject: [clinthuffman/Clue] Does EmailReportTo work as advertised? (Issue #2)

Love the tool & just using it for the first time here. I have configured EmailReportTo and entered an email address - but I can't see where I can configure (say) an SMTP server, not can I see anywhere in the code where the EmailReportTo value is processed.

Is this feature working?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fclinthuffman%2FClue%2Fissues%2F2&data=04%7C01%7C%7Cad0546eaefaa49e483c408d9b4e5f594%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637739722022241890%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=2nXdL0tz1c1aFvE6tpwmB6NWlsN7RRNjZS%2B59GThGt8%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEXDYVFAQAJSWBAUXPSKL6DUOZC6RANCNFSM5JE4JBXA&data=04%7C01%7C%7Cad0546eaefaa49e483c408d9b4e5f594%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637739722022241890%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=jxPtN78rGdQHjKiczxiuOWYmve3%2B6KXh2v%2BRO1DI2OA%3D&reserved=0. Triage notifications on the go with GitHub Mobile for iOShttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04%7C01%7C%7Cad0546eaefaa49e483c408d9b4e5f594%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637739722022251851%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=Yk73mWHv4Ke4TKfIZO8GEkYONKXKnzSJnCwWQ25jYNU%3D&reserved=0 or Androidhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04%7C01%7C%7Cad0546eaefaa49e483c408d9b4e5f594%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637739722022251851%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=NLtg7DHGaGdoFOUa%2FEJyHfcqBRMR9IgZBbYoBMLYqok%3D&reserved=0.

Borgquite commented 2 years ago

Thanks Clint. If you ever get the opportunity to implement it that would be extremely helpful. In the mean time it would help if the dialog box was removed or updated to reflect that it's informational only, as it does give the impression that it's working!

(The 'Clue Usage Guide.docx' gives the same impression too).

I'm loving the tool though, and using it has also meant I discovered the Windows Performance Analyzer which I was extremely impressed with too!

Thank you so much for contributing this to the community.

clinthuffman commented 2 years ago

The email address still serves the purpose of letting the person doing the analysis know who the data came from which will help when we can offer automated analysis of the traces.

In the meantime, if you want to suppress all of the dialogs during setup, you can set/fill in the values in the first line of the config.xml - or leave them blank for users to fill in during setup. For example, the Email Address dialog can be suppressed by setting EmailReportTo attribute in the config.xml. Just make it a fake or real email address if you want because it does check it as a regular expression.

I can’t provide any details, but I’ll say that Microsoft, the company that I work for, has taken an interest in this kind of data collection of performance issues. CLUE will continue to be free, but we are thinking of redesigning and updating it again including automated analysis of the data – something I’ve been dreaming about for years.

Thank you, Clint Huffman @.***)

From: Chris @.> Sent: Thursday, December 2, 2021 2:33 AM To: @.> Cc: Clint @.>; @.> Subject: Re: [clinthuffman/Clue] Does EmailReportTo work as advertised in 1.1.6? (Issue #2)

Thanks Clint. If you ever get the opportunity to implement it that would be extremely helpful. In the mean time it would help if the dialog box was removed or updated to reflect that it's informational only, as it does give the impression that it's working!

(The 'Clue Usage Guide.docx' gives the same impression too).

I'm loving the tool though, and using it has also meant I discovered the Windows Performance Analyzer which I was extremely impressed with too!

Thank you so much for contributing this to the community.

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fclinthuffman%2FClue%2Fissues%2F2%23issuecomment-984499979&data=04%7C01%7C%7C2772a0c4d22649f4f8be08d9b57f3858%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637740380270971301%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=G6mgFajHlDERCK9WKu7LTXA2zvjbBJfUyddCz4gWYpw%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEXDYVE7CLWEBQFP7FLQOKDUO5DQTANCNFSM5JE4JBXA&data=04%7C01%7C%7C2772a0c4d22649f4f8be08d9b57f3858%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637740380270971301%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=1VbRvCYY5EWx03sxo%2BA291pZjtK85e%2BXGIg5h50Cjs8%3D&reserved=0. Triage notifications on the go with GitHub Mobile for iOShttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04%7C01%7C%7C2772a0c4d22649f4f8be08d9b57f3858%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637740380270971301%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=ue4CIzY%2BYXJSQxt07tXGM%2FGlk%2FFyteNOT7WvUE%2BC0sQ%3D&reserved=0 or Androidhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04%7C01%7C%7C2772a0c4d22649f4f8be08d9b57f3858%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637740380270971301%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=gOFquu1occ5y1CW7SDBS9C1Q%2F6jofy%2BQctLmW0OSEq0%3D&reserved=0.