Skip to content

Clarify exception resolver request logging metadata#15811

Open
jamesfredley wants to merge 1 commit into
8.0.xfrom
fix/request-logging-metadata-default
Open

Clarify exception resolver request logging metadata#15811
jamesfredley wants to merge 1 commit into
8.0.xfrom
fix/request-logging-metadata-default

Conversation

@jamesfredley

@jamesfredley jamesfredley commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

The Problem

The runtime default for grails.exceptionresolver.logRequestParameters is environment-dependent: request-parameter logging is enabled in development and disabled everywhere else.

The Spring configuration metadata, however, advertised a single unconditional defaultValue. IDEs and config-metadata consumers that read that value would present a default that does not match what actually happens at runtime outside development.

The Fix

Make the published metadata tell the truth instead of a misleading constant.

  • Removed the unconditional metadata defaultValue.
  • Documented the real environment-dependent behavior (enabled in development, disabled otherwise) directly in the property description.

Files

  • grails-web-core/.../META-INF/spring-configuration-metadata.json - drop the hard-coded default, describe the dev/non-dev behavior.

Testing

  • :grails-web-core:processResources succeeds and the emitted metadata for grails.exceptionresolver.logRequestParameters has no defaultValue and carries the environment-dependent description.
  • LSP diagnostics clean for the metadata JSON.

The runtime default is environment-dependent: request parameters log in development and remain disabled otherwise. Remove the unconditional metadata default and describe the actual behavior.

Assisted-by: Hephaestus:openai/gpt-5.5 oracle codex-review
Copilot AI review requested due to automatic review settings July 2, 2026 02:27

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Spring configuration metadata for grails.exceptionresolver.logRequestParameters to avoid advertising an unconditional default when the runtime default is environment-dependent, improving accuracy for IDE/config metadata consumers.

Changes:

  • Removed the unconditional defaultValue from grails.exceptionresolver.logRequestParameters metadata.
  • Updated the property description to document the development vs non-development default behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov

codecov Bot commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.4766%. Comparing base (66cd24c) to head (4febf42).

Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                 @@
##                8.0.x     #15811        +/-   ##
==================================================
- Coverage     49.4842%   49.4766%   -0.0076%     
+ Complexity      16697      16695         -2     
==================================================
  Files            1947       1947                
  Lines           92474      92474                
  Branches        16152      16152                
==================================================
- Hits            45760      45753         -7     
- Misses          39606      39612         +6     
- Partials         7108       7109         +1     

see 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@testlens-app

testlens-app Bot commented Jul 2, 2026

Copy link
Copy Markdown

✅ All tests passed ✅

🏷️ Commit: 4febf42
▶️ Tests: 43691 executed
⚪️ Checks: 45/45 completed


Learn more about TestLens at testlens.app.

@jdaugherty jdaugherty left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change should be made against 7.0.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants