GluuFederation / flex

Janssen + AdminUI + Casa
23 stars 14 forks source link

fix(admin-ui): updating asset showing 500 error #1861

Open manojs1978 opened 1 week ago

manojs1978 commented 1 week ago

Describe the bug update asset giving 500 error

To Reproduce Steps to reproduce the behavior:

  1. install flex 5.1.6.nightly
  2. login to admin ui
  3. click asset in admin
  4. upload new asset of png file in jans-auth
  5. edit uploaded asset and modify file , name and relative service
  6. save changes
  7. See error

Expected behavior should not get any error

Screenshots If applicable, add screenshots to help explain your problem. image

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context Add any other context about the problem here.

https://github.com/user-attachments/assets/6de03db1-9c2b-42d6-8226-f41b39d0a9b7

pujavs commented 3 days ago

@manojs1978 for jans-lock the config is avialable only for asset type .jar. You can confirm this either from any UI or from source code -> https://github.com/JanssenProject/jans/blob/main/jans-linux-setup/jans_setup/templates/jans-config-api/dynamic-conf.json#L78-L296

Request you to please try uploading a jar file. I will also update the error message to indicate the same.

pujavs commented 3 days ago

Made error message descriptive via PR [9919] (https://github.com/JanssenProject/jans/pull/9919)

pujavs commented 2 days ago

@manojs1978, request you to please verify and confirm

manojs1978 commented 2 days ago

ok checking once changes in latest build