jhl221123 opened a new pull request, #3691: URL: https://github.com/apache/logging-log4j2/pull/3691
This PR addresses issue #3176 by ensuring scripts defined in the global `<Scripts>` container have explicit, non-blank names. It also introduces a clearer distinction between user-defined names and internal script identifiers. **Key Changes:** * `AbstractScript` now provides `getName()` strictly for user-configured names, which can be null or blank. It also introduces `getId()` to consistently return a non-blank internal identifier, falling back to `toString()` if no explicit name is set. * `ScriptsPlugin` now throws a `ConfigurationException` if any script defined directly under the global `<Scripts>` element lacks an explicit name. * `ScriptManager` and various script-using components were updated. They now utilize the `script.getId()` for internal referencing. * Added unit tests for `AbstractScript` and `ScriptsPlugin`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@logging.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
