Project

General

Profile

Bug #38550

Add credential validation to Cloud Credentials in middleware

Added by Timothy Moore II 7 months ago. Updated 8 days ago.

Status:
Done
Priority:
No priority
Assignee:
Vladimir Vinogradenko
Category:
Middleware
Target version:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

Testing with FreeNAS Mini updated to FreeNAS-11.2-MASTER-201807160837:

Go to System/Cloud Credentials/Add. Choose “Amazon S3” as the “Provider” and enter invalid data in each remaining field. Click “Save”.

Go to Tasks/Cloud Sync/Add and Create a new Cloud Sync Task. Add invalid S3 Credential. FreeNAS attempts to validate the credential, fails, and displays an error. Should this validation also be done when saving the new credential in System/Cloud Credentials? This will prevent the user from getting farther into the process with improperly configured credentials.


Related issues

Related to FreeNAS - Feature #39872: Can we use a more elegant warning than the modal window in the New GUI if a cloud sync task being edited has an authetication problemUnscreened
Copied to FreeNAS - Bug #51700: Add Credential validation to System/Cloud Credentials in new UIIn Progress

Associated revisions

Revision 72f53414 (diff)
Added by Vladimir Vinogradenko 4 months ago

Add Credential validation to System/Cloud Credentials

Ticket: #38550

Revision fb11ec56 (diff)
Added by Vladimir Vinogradenko 4 months ago

Add Credential validation to System/Cloud Credentials

Ticket: #38550

History

#1 Updated by Dru Lavigne 7 months ago

  • Target version changed from Backlog to 11.2-RC2

#2 Updated by Dru Lavigne 7 months ago

  • Target version changed from 11.2-RC2 to 11.2-BETA3

#3 Avatar?id=14398&size=24x24 Updated by Kris Moore 7 months ago

  • Target version changed from 11.2-BETA3 to Backlog

#5 Updated by Erin Clark 7 months ago

  • Assignee changed from Erin Clark to Lola Yang

#6 Updated by Lola Yang 4 months ago

  • Category changed from GUI (new) to Middleware
  • Assignee changed from Lola Yang to William Grzybowski

We need more middleware work to test the validation of the credentials.

#7 Updated by William Grzybowski 4 months ago

  • Assignee changed from William Grzybowski to Vladimir Vinogradenko
  • Target version changed from Backlog to 11.3

#8 Updated by Bug Clerk 4 months ago

  • Status changed from Unscreened to In Progress

#9 Updated by Bug Clerk 4 months ago

  • Status changed from In Progress to Ready for Testing

#10 Updated by Vladimir Vinogradenko 4 months ago

  • Status changed from Ready for Testing to Unscreened
  • Assignee changed from Vladimir Vinogradenko to Lola Yang

New method: cloudsync.credentials.verify

Parameters:

{
    "provider": "S3",
    "attributes": {...}
}

Returns

{
    "valid": true,
}

or

{
    "valid": false,
    "error": "Some reason",
}

#11 Updated by Vladimir Vinogradenko 4 months ago

Proposed logic is:

1. Client checks that credentials are valid before saving them

2. If they are not, warning is displayed but saving is still allowed

#12 Updated by Dru Lavigne 4 months ago

  • Subject changed from Add Credential validation to System/Cloud Credentials to Add Credential validation to System/Cloud Credentials to middleware
  • Status changed from Unscreened to Ready for Testing
  • Assignee changed from Lola Yang to Vladimir Vinogradenko
  • Needs Merging changed from Yes to No

#13 Updated by Dru Lavigne 4 months ago

  • Subject changed from Add Credential validation to System/Cloud Credentials to middleware to Add Credential validation to System/Cloud Credentials in middleware

#14 Updated by Dru Lavigne 4 months ago

  • Copied to Bug #51700: Add Credential validation to System/Cloud Credentials in new UI added

#15 Updated by Lola Yang 4 months ago

  • Related to Feature #39872: Can we use a more elegant warning than the modal window in the New GUI if a cloud sync task being edited has an authetication problem added

#16 Updated by Dru Lavigne about 2 months ago

  • Subject changed from Add Credential validation to System/Cloud Credentials in middleware to Add credential validation to Cloud Credentials in middleware

#17 Updated by Dru Lavigne 29 days ago

  • Target version changed from 11.3 to 11.3-BETA1

#18 Updated by Dru Lavigne 8 days ago

  • Status changed from Ready for Testing to Done
  • Needs QA changed from Yes to No

Also available in: Atom PDF