regression: when migrating from 5 to 6, a datasource with an empty password throws an admin error

Description

using a mysql 8.0.33 datasource in admin

  1. open the datasource, save it, shows ups as ok

  2. select the datasource and verify it, shows error

  3. or access the database, an error relating to password is shown

  4. open the datasource and save it again,, it says ok

  5. repeat 2 or 3, same error

lucee is running in multi context mode

this is using preside (which hides datasource errors OOTB https://presidecms.atlassian.net/browse/PRESIDECMS-2537 )

Environment

None

Attachments

3
  • 02 Jun 2023, 07:12 am
  • 30 May 2023, 04:15 pm
  • 30 May 2023, 10:55 am

Activity

Show:

Zac Spitzer 2 June 2023 at 14:18

https://github.com/lucee/Lucee/commit/b68f149cc81e524841ff605692f06eea77a17535


(to test simply delete the password key from a datasource in cfconfig.json

Zac Spitzer 2 June 2023 at 07:12

https://github.com/lucee/Lucee/pull/2094

added a null check, also I improved the verify to show the error inline, rather than only as a tooltip

Zac Spitzer 30 May 2023 at 16:13
Edited

ok problem is with an upgrade from 5 to 6, a migrated datasource without a password causes the above error

Zac Spitzer 30 May 2023 at 13:55
Edited

in the datasource definition in CFconfig.json, the password key was missing completely, hence the null?

can’t repo via the admin UI, need to investigate how to repo. deleting the password from the datasource definition manually repros

Michael Offner 30 May 2023 at 13:54

the issue only happens when you have a datasource with the same name in the Lucee Server and web context with different passwords?

I was no able to reproduce the issue, based on the information in the description. i was following the instruction 1 to 1, but was not able to reproduce the issue.

Fixed

Details

Assignee

Reporter

Priority

Fix versions

New Issue warning screen

Before you create a new Issue, please post to the mailing list first https://dev.lucee.org

Once the issue has been verified, one of the Lucee team will ask you to file an issue

Sprint

Affects versions

Created 29 May 2023 at 13:55
Updated 9 June 2023 at 15:32
Resolved 9 June 2023 at 15:32

Flag notifications