microsoft / sarif-visualstudio-extension

SARIF Microsoft Visual Studio Viewer Extension
MIT License
46 stars 19 forks source link

Using registry keys to limit devcanvas usage to just team members #613

Closed edkazcarlson-ms closed 12 months ago

edkazcarlson-ms commented 1 year ago

This is a temporary change that will allow the DevCanvas team to make sure our changes do not affect the wider audience while also allowing us to merge back into main in the near future.

Our current plan for releasing this feature branch is:

  1. Commit this PR to the feature branch
  2. Get access to the app insights cluster from @michaelcfanning
  3. Set up some telemetry points around our auth stack/plugin in order to validate that it is working
  4. Merge this feature branch into main in order to guarantee nothing gets changed or refactored in main that would cause headaches in the feature branch
  5. When we feel ready, remove the changes in this PR that check for the devcanvas reg key to release to the wider MSFT audience

While we are not going to commit it to the repo, the following is a simple .reg file that can be used to turn on the reg key flag that we are using in this PR.

Windows Registry Editor Version 5.00

[HKEY_CURRENT_USER\Software\Microsoft\VisualStudio\devcanvas]
"DevCanvasInsider"="True"

To use this .reg file, copy the content to a file and make sure the file extension is .reg. Once you have the file saved, double click it and it should automatically apply the registry key changes needed to active the plugin.

ShiningMassXAcc commented 1 year ago

Tagging @michaelcfanning directly - we are still blocked here generally behind access to application insights.

We want to do the right engineering practice and use Observability to track successes/failures in production given that most interactions should not result in a crash (which we get automatic telemetry for).