The GitHub Actions job "Licensing - RAT Report" on 
grails-core.git/7.1.x-grailsutil-honor-filterer-config has succeeded.
Run started by GitHub user codeconsole (triggered by codeconsole).

Head commit for run:
cdffb8f381c2095dfb16af81337b79677737cbf0 / Scott Murphy Heiberg 
<[email protected]>
GrailsUtil: bootstrap-hook init via GrailsExceptionResolver

Address review feedback on #15666:

- Replace the lazy resolveStackFilterer/findApplicationQuietly pattern with
  a deterministic bootstrap-hook: GrailsExceptionResolver.setGrailsApplication
  calls the new public GrailsUtil.initializeStackFilterer(application) during
  Spring bean wiring. Hot path collapses to a single volatile read.

- Restore the single-instance invariant of the prior static final field via a
  static final FALLBACK_FILTERER sentinel. The volatile stackFilterer is
  initialised to FALLBACK_FILTERER so CLI, tests that don't boot a context,
  and plain main() usage continue to work with one shared instance.

- Drop findApplicationQuietly and the surrounding try/catch. Holders no
  longer needs to be consulted from GrailsUtil at runtime; the resolver
  pushes the application in during its own wiring.

GrailsUtilStackFiltererSpec rewritten to cover the new shape: fallback
before init, null-application no-op, configured-class wiring,
logFullStackTraceOnFilter propagation, and last-write-wins on repeat init.

Existing GrailsExceptionResolverSpec, StackTraceFiltererSpec and
GrailsUtilTests continue to pass; checkstyle clean.

Docs in upgrading71x.adoc and loggingFullStackTraces.adoc updated to
describe the bootstrap-hook wiring instead of the prior lazy resolution.

Report URL: https://github.com/apache/grails-core/actions/runs/26627848177

With regards,
GitHub Actions via GitBox

Reply via email to