Document query parameter parsing bug in PR #1217

[Copilot]

PR eclipse-equinox#1217 attempts to add query parameter support to bundle resource URLs but the implementation is broken. All 8 tests fail with url.getQuery() returning null.

Issue

The parseURL() method in BundleResourceHandler.java has extraction logic for query/fragment (lines 100-118) but it doesn't work:

String query = url.getQuery();  // Returns null
int queryIdx = spec.indexOf('?', pathIdx);  // Finds '?' but extraction fails
// ... extraction logic ...
setURL(url, ..., query, ref);  // Query remains null after this

Test failures:

URL urlWithQuery = new URL(entry.toExternalForm() + "?param=value");
assertEquals("param=value", urlWithQuery.getQuery());  // FAILS: expected:<param=value> but was:<null>

Investigation Results

Attempted fixes:

• Parsing from url.getQuery() (pre-parsed by Java URL class) → null
• Parsing from spec substring → query not found in spec
• Parsing from original str parameter → still fails
• Combination approaches → no success

Root cause unclear. Likely related to OSGi's PlurlStreamHandler/PlurlSetter mechanism or how Java URL class passes spec to custom protocol handlers.

Review Comment Status

The review asks for tests to verify "both URL parsing AND openConnection + reading content". Cannot add openConnection tests until parsing works. Tests at lines 180-225 already verify both, but fail due to parsing bug.

Next steps: Debug and fix query parameter extraction, then address review comments.

See INVESTIGATION_NOTES.md for detailed analysis.

Original prompt

Please address the review comments from here eclipse-equinox#1217 (review)

---

💡 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.