[Hardik K (hardikkaurani)]

Summary

Fixes #2235

TextParseNodeFactory.getParseNode() and TextSerializationWriterFactory.getSerializationWriter() used strict string equality (!contentType.equals(validContentType)) to validate the content type against text/plain.

This caused both factories to reject valid content types that contain MIME parameters (such as text/plain; charset=utf-8 or text/plain; charset=UTF-8) or use case variants (e.g. TEXT/PLAIN or Text/Plain; charset=utf-8).

Per RFC 2045 §5.1, media types and subtypes are case-insensitive and parameter-agnostic for type identification.

Changes

• Commit 1 (fix): Updated TextParseNodeFactory and TextSerializationWriterFactory to strip MIME parameters (split(";", 2)[0].trim()) and compare against the valid content type case-insensitively (equalsIgnoreCase), matching the pattern implemented in MultipartSerializationWriterFactory (fix(multipart): support content-type parameters and case-insensitivity in MultipartSerializationWriterFactory #2229).
• Commit 2 (tests): Added TextParseNodeFactoryTest with unit and parameterized regression tests covering valid text/plain, content types with charset parameters, uppercase, mixed case, invalid types, empty strings, and delimiter-only inputs.
• Commit 3 (tests): Added TextSerializationWriterFactoryTest with corresponding unit and parameterized regression tests for the writer factory.

Verification

• spotlessCheck / spotlessApply: PASS
• compileJava / compileTestJava: PASS
• spotbugsMain / spotbugsTest: PASS
• :components:serialization:text:check: PASS (100% tests passing)
• :components:bundle:check: PASS

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

[Hardik K (hardikkaurani)]

Vincent Biret (@baywet) Opened this PR to address #2235 by aligning TextParseNodeFactory and TextSerializationWriterFactory with RFC 2045 §5.1, mirroring the fix recently merged for MultipartSerializationWriterFactory (#2229).

Previously, both text factories used strict string equality (!contentType.equals("text/plain")), causing them to reject valid text payloads specifying standard MIME parameters (e.g., text/plain; charset=utf-8) or case variants (TEXT/PLAIN).

The PR is structured across 3 focused commits:

1. fix(serialization): Strips MIME parameters and compares against text/plain case-insensitively across both factories.
2. test(serialization): Adds TextParseNodeFactoryTest with unit and parameterized coverage for parameters, casing, invalid types, empty strings, and delimiters.
3. test(serialization): Adds TextSerializationWriterFactoryTest with corresponding unit and parameterized test coverage.

Verified locally: spotlessCheck, spotbugsMain, spotbugsTest, and :components:serialization:text:check pass cleanly.

[Copilot]

🟢 Approval recommended

No active review issues remain for the changes supporting parameterized and case-varied text/plain media types.

No open issues 🎉

_{Review effort: Balanced}