search

microsoft / VoTT

Visual Object Tagging Tool: An electron app for building end to end Object Detection Models from Images and Videos.
MIT License
4.3k stars 837 forks source link

Confusing language in Connection Settings #899

Open toddhaugen opened 5 years ago

toddhaugen commented 5 years ago

Describe the bug Connectino settings page requires the user enter a SAS. The SAS page has three potentially viable value for the SAS box and the assistance text under the entry box does not clarify.

To Reproduce Steps to reproduce the behavior:

  1. Go to home page and click on new project
  2. Click on click on add connection
  3. Scroll down to provider and select Azure Bloc Storage
  4. this formats the page to accept an "SAS" which is described as a Shared Access Signature used to authenticate to the blob storage account.
  5. Open Azure Portal, navigate to a your storage account and click on the SAS blade.
  6. if you do not already configured SAS so so with any values.
  7. the issue is the user is given the choice of three items that might be relevant because the terminology does not align:
    • Connectino String, the common value used with azure storage
    • SAS Token
    • Blob Service SAS URL

Expected behavior Data Scientists not familiar with the Azure security model will not know which value to pick. Changing the SAS test box label to say SAS Token will add significant clarity combined with an update to the assistance text under the text box to say something like, "Shared Access Signature Token used to authenticate to the blob storage account which is retrieved from the SAS blade under your blob storage account."

Screenshots From the connection settings page in VoTT image From the SAS blade in azure portal: image