Skip to content

Correct historical context of deserialization FAQ - #43

Merged
ppkarwasz merged 2 commits into
mainfrom
fix-deserialization-historical-context
Aug 27, 2026
Merged

Correct historical context of deserialization FAQ#43
ppkarwasz merged 2 commits into
mainfrom
fix-deserialization-historical-context

Conversation

@ppkarwasz

Copy link
Copy Markdown
Member

The security FAQ claimed that the SocketServer receiver was never reintroduced into the Log4j 2 production codebase. This is inaccurate: log4j-core contained TCP and UDP socket servers inherited from Log4j 1 up to and including the 2.8.x releases. As any endpoint consuming serialized data, they were dangerous to use and resulted in CVE-2017-5645. The vulnerability was patched in 2.8.2 by introducing FilteredObjectInputStream as a Java 7 equivalent of the ObjectInputFilter API introduced in Java 9, but since the fix could not cover all cases the socket servers were removed in 2.9.0.

This change corrects the section and expands it with this history, cross-linking the CVE-2017-5645 entry on the security page.

🤖 Generated with Claude Code

https://claude.ai/code/session_01GeV6uBXR77uZVnPAUUjF6n

The security FAQ claimed that the `SocketServer` receiver was never reintroduced into the Log4j 2 production codebase. This is inaccurate: `log4j-core` contained TCP and UDP socket servers inherited from Log4j 1 up to and including the 2.8.x releases. As any endpoint consuming serialized data, they were dangerous to use and resulted in CVE-2017-5645. The vulnerability was patched in 2.8.2 by introducing `FilteredObjectInputStream` as a Java 7 equivalent of the `ObjectInputFilter` API introduced in Java 9, but since the fix could not cover all cases the socket servers were removed in 2.9.0.

This change corrects the section and expands it with this history, cross-linking the CVE-2017-5645 entry on the security page.

Assisted-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GeV6uBXR77uZVnPAUUjF6n
@ppkarwasz
ppkarwasz enabled auto-merge (squash) August 27, 2026 05:09

@FreeAndNil FreeAndNil 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.

LGTM

@ppkarwasz
ppkarwasz merged commit 2cd27ed into main Aug 27, 2026
5 checks passed
@ppkarwasz
ppkarwasz deleted the fix-deserialization-historical-context branch August 27, 2026 06:49
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.

2 participants