Skip to content

Conversation

@jimblye
Copy link
Member

@jimblye jimblye commented Nov 27, 2025

Reintroduces #33455, but ignores zero-width characters. These characters don't interfere with the parsing and cannot be mistaken for a missing "<" on an element. These characters might be inserted during copy/paste operation:
U+200B: Zero Width Space
U+200C: Zero Width Non-Joiner
U+200D: Zero Width Joiner
U+FEFF: Zero Width No-Break Space (BOM)
U+2060: Word Joiner

The original PR detected and displayed an error message for 3 existing server.xml files which contained some of these invisible characters:

com.ibm.ws.request.timing.monitor_fat/publish/servers/RequestTimingFeatureWithEventFeature/server.xml
Line 34:  extraneous unicode character
io.openliberty.microprofile.openapi.2.0.internal_fat/publish/servers/OpenAPIMergeWithServerXMLTestServer/server.xml
Line 27:extraneous unicode character
io.openliberty.microprofile.openapi.2.0.internal_fat/publish/servers/OpenAPIWarningMessageTestServer/server.xml
Line 23:extraneous unicode character

@jimblye jimblye self-assigned this Nov 27, 2025
@jimblye jimblye force-pushed the 33453-server-xml-configuration-error-not-detected-2 branch from 3303317 to 528c6b6 Compare November 29, 2025 16:35
@jimblye
Copy link
Member Author

jimblye commented Nov 30, 2025

Found a server.xml file in WS-CD-Open having a back tick ` between elements.
#build (view Open Liberty Personal Build - ❌ completed with errors/failures.)

Note: Target locations of links might be accessible only to IBM employees.

@jimblye jimblye changed the title Server.xml configuration error not detected. Reintroduce - Server.xml configuration error not detected. Dec 4, 2025
@jimblye jimblye force-pushed the 33453-server-xml-configuration-error-not-detected-2 branch from 453a957 to 9f08987 Compare December 5, 2025 23:10
@jimblye
Copy link
Member Author

jimblye commented Dec 5, 2025

#build
(view Open Liberty Personal Build - ❌ completed with errors/failures.)
#libby

Note: Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link
Member

LibbyBot commented Dec 5, 2025

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 2 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

  • 1 messages files were changed and need an L2 review.

  • @OpenLiberty/message-reviewer Please review.

    • dev/com.ibm.ws.config/resources/com/ibm/ws/config/internal/resources/ConfigMessages.nlsprops
  • 1 NLS files were changed and need an ID review.

  • @OpenLiberty/message-reviewer Please review.

    • dev/com.ibm.ws.config/resources/com/ibm/ws/config/internal/resources/ConfigMessages.nlsprops

@OpenLiberty OpenLiberty deleted a comment from LibbyBot Dec 5, 2025
@OpenLiberty OpenLiberty deleted a comment from LibbyBot Dec 5, 2025
@OpenLiberty OpenLiberty deleted a comment from LibbyBot Dec 5, 2025
@OpenLiberty OpenLiberty deleted a comment from LibbyBot Dec 5, 2025
@OpenLiberty OpenLiberty deleted a comment from LibbyBot Dec 5, 2025
@OpenLiberty OpenLiberty deleted a comment from LibbyBot Dec 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Server.xml configuration error not detected.

3 participants