search

dotCMS / core

Headless/Hybrid Content Management System for Enterprises
http://dotcms.com
Other
866 stars 467 forks source link

App configuration export needs better error messaging #25568

Open jdcmsd opened 1 year ago

jdcmsd commented 1 year ago

Problem Statement

It's easy to see this when attempting to export App configurations:

image

The error is most likely due to the length of the password provided, but the message gives no clue of this.

There are also sometimes permission limitations — e.g., a non-admin can't export them at all. Even if you give a good password, you might see this appear in the logs:

[19/07/23 17:59:28:754 UTC]  INFO util.SecurityLogger: class com.dotcms.rest.api.v1.authentication.ResponseUtil : Only Admins are allowed to perform an export operation. -- ip:###.###.###.###,user:XXXXXXX [ID: X][email:X.X@X.com]

But, again, no sign of it on the UI. We should change that!

Steps to Reproduce

  1. Hop on to Demo
  2. Go to Settings -> Apps
  3. Attempt to export all app configurations
  4. Open browser Dev Tools and browse to the Network tab to monitor the API calls
  5. Set a password of <14 (or >32) characters in length and attempt to export
  6. See error as pictured above
  7. See Error message under the failed export call image

Similarly: "Login As" a non-admin, and repeat the above steps with an appropriate-length password

Acceptance Criteria

The error messages in the UI should be clearer as to what went wrong.

dotCMS Version

master (7/19/2023)

Proposed Objective

Technical User Experience

Proposed Priority

Priority 3 - Average

github-actions[bot] commented 1 year ago

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 30 days.