Skip to content

Fix server crash when File-Based Debug Logger is enabled#602

Open
Copilot wants to merge 3 commits intomasterfrom
copilot/fix-server-crash-file-debug-logger
Open

Fix server crash when File-Based Debug Logger is enabled#602
Copilot wants to merge 3 commits intomasterfrom
copilot/fix-server-crash-file-debug-logger

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 30, 2026

  • Add Portions Copyright 2026 3A Systems, LLC. to LdapMessages.java
  • Add Portions Copyright 2026 3A Systems, LLC. to LoggerConfigManager.java

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

When the debug logger is enabled, LoggerConfigManager sets root JUL level
to FINEST which makes Grizzly's DefaultFilterChain.executeFilter() check
isLoggable(FINEST)=true and log the FilterChainContext object.

In Grizzly 3.0.1, FilterChainContext.toString() has a bytecode bug where
it does a checkcast to char[] on the message field. When the message is
a LdapResponseMessage, the cast fails with ClassCastException that
propagates through the RxJava pipeline causing a server crash.

Fix: Limit org.glassfish.grizzly JUL logging to FINE level so that
isLoggable(FINEST) returns false, preventing the broken code path.

Also add toString() to LdapResponseMessage for better diagnostics.

Agent-Logs-Url: https://github.com/OpenIdentityPlatform/OpenDJ/sessions/73a4bf17-653a-4ea6-b379-6a220c9059db

Co-authored-by: vharseko <6818498+vharseko@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix server crash when file-based debug logger is enabled Fix server crash when File-Based Debug Logger is enabled Mar 30, 2026
Copilot AI requested a review from vharseko March 30, 2026 09:33
@vharseko vharseko requested a review from maximthomas March 30, 2026 09:35
@vharseko vharseko marked this pull request as ready for review March 30, 2026 09:35
Copy link
Copy Markdown
Contributor

@maximthomas maximthomas left a comment

Choose a reason for hiding this comment

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

Tested locally, works fine with File-Based Debug Logger enabled

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ver 5.x Server Crash When File-Based Debug Logger is enabled

3 participants