Skip to content

Handle referencing schemas through UNC paths#1782

Merged
angelozerr merged 1 commit into
eclipse-lemminx:mainfrom
datho7561:handle-schemas-accessed-through-unc
Jun 5, 2026
Merged

Handle referencing schemas through UNC paths#1782
angelozerr merged 1 commit into
eclipse-lemminx:mainfrom
datho7561:handle-schemas-accessed-through-unc

Conversation

@datho7561
Copy link
Copy Markdown
Contributor

  • When downloading external resources is disabled, don't attempt to download non-local schema files referenced using UNCs
    • eg. file://example.org/schemas/my-schema.xsd, //example.org/schemas/my-schema.xsd, file:////example.org/schemas/my-schema.xsd
  • Use the schema cache for resources referenced through UNCs
    • Unless the authority is localhost eg. //localhost/C$/Users/Mira/my-schema.xsd
  • Fix a minor bug where the error range is wrong for schemas referenced through a UNC starting with // in <?xml-model href="...

@datho7561 datho7561 added the bug Something isn't working label Jun 4, 2026
- When downloading external resources is disabled, don't attempt
  to download non-local schema files referenced using UNCs
- Use the schema cache for resources referenced through UNCs
  - eg. `file://example.org/schemas/my-schema.xsd`,
    `//example.org/schemas/my-schema.xsd`,
    `file:////example.org/schemas/my-schema.xsd`
  - Unless the authority is `localhost` eg.
    `//localhost/C$/Users/Mira/my-schema.xsd`
- Fix a minor bug where the error range is wrong for schemas referenced through
  a UNC starting with `//` in `<?xml-model href="...`

Signed-off-by: David Thompson <davidethompson@me.com>
@datho7561 datho7561 force-pushed the handle-schemas-accessed-through-unc branch from e4145d9 to 97d3841 Compare June 4, 2026 21:29
@angelozerr
Copy link
Copy Markdown
Contributor

LGTM thanks @datho7561 !

@angelozerr angelozerr added this to the 0.32.0 milestone Jun 5, 2026
@angelozerr angelozerr merged commit 6a5523e into eclipse-lemminx:main Jun 5, 2026
6 checks passed
@datho7561 datho7561 deleted the handle-schemas-accessed-through-unc branch June 5, 2026 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants