cflog doesn't write application name to log

Description

see the test case for https://luceeserver.atlassian.net/browse/LDEV-3264

`<cflog application="true" type="Information" text="blah blah">`

Environment

None

Activity

Show:
Pothys - MitrahSoft
March 9, 2021, 1:16 PM

this issue was started regression by the ticket LDEV-3001. Before that, writelog logs an application name only when application=true So I added a fix to log application name in logs even attribute application is not set. I think this causes the regression. Now this works fine with fixed versions. So I mark this as deployed.

Zac Spitzer
March 8, 2021, 4:04 PM

that may be true, but I think this regression was caused by the 5.3.8 logging changes

it did previously work, there was a whole series of builds after those changes which didn’t produce a snapshot, so it isn’t easy to work out what caused this regression

Michael Offner
March 8, 2021, 3:51 PM

this is marked as a regression, but the “PR above does not revert any change made in the past, it changes the default value of an argument to a value it never had.

Pothys - MitrahSoft
February 8, 2021, 8:20 AM

I've checked this ticket and confirmed the application name not rendered in the log with application=true. I added a fix for this ticket

Pull Request: https://github.com/lucee/Lucee/pull/1180

Fixed
Your pinned fields
Click on the next to a field label to start pinning.

Assignee

Pothys - MitrahSoft

Reporter

Zac Spitzer

Priority

New

Fix versions