Reintroduced Analyse button and details pages #7
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- collapse long stack traces - wrap long lines - add search and reload to view log - renamed details to analysis - fixed crash viewing individual detail record (serializeJson instead of serialize) - added page titles - always include the plugin stylesheet for each template - fixed sortable header colors (clashed with bg)
- getLog function - extra styles
- also include context url in dropdown
the script tag is added using cfhtmlbody, so it's useful to be able to see where it's coming from
- converted cfc to script - split code out into a few different cfc's - move javascript into seperate files - use org.apache.commons.io.input.ReversedLinesFileReader
refactored log parsing logs are tailed inline refreshing
- limit parsing to a maximum of 2000 logs (not lines) - after a new set of logs is loaded from the server, the displayed list is trimmed after the new logs are inserted into the dom
- view all files in a single combined view - avoid leaving open file handles - heaps more refactoring - added an auto refresh function
massive performance improvement!
- switched from GUID to lucee-log-viewer
sigh, lucee doesn't support nice names for extension ids
- read from datasource (all data) - bulk admin updater for log config - only parse stacks for returned logs #2
add a GHA
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Reintroduced Analyse button that groups same errors
Added the list function from loganalyzer that groups all the same error and also change it to deal with error messages added in JSON format
Updated the detail page
Removed link from header for Log Storage table on the admin page as it was conflicting with the log listing, plus it's not necessary to have it there
Update LogGateway.cfc L#91 to return all log files (log so it includes application.log as well as application.log.1.bak)
Updated the return action when deleting log file and also on some of the back buttons
logViewer.zip