Bug #5952
closed![Author: Nico Kutscherauer](https://www.gravatar.com/avatar/e854da729f4f7bfa70b1c3c023f60e53?rating=PG&size=50&default=https%3A%2F%2Fassets.plan.io%2Fimages%2Fdefault_avatar.png)
unparsed-text-available throws an un-catchable IllegalCharsetNameException
100%
Description
If I try to read the following non-wellformed XML document with the function unparsed-text()
:
<?xml version="1.0" encoding="foo"?>
<foo/>
I get an IllegalCharsetNameException. It was suprising to me, but understandable. The problem is that I can't catch this error. If I try to check before with unparsed-text-available()
if the document is readable I get the same error.
Even with <xsl:try>
I can not prevent that the process is terminated by this exception.
This is not the desired behaviour, is it?
(I tried it in several versions of Saxon-HE 10.6, 11.5, 12.1 - all the same behavior.)
Updated by Michael Kay over 1 year ago
Thanks for reporting it.
The spec doesn't give a definitive answer what should be done here, but it certainly shouldn't be an uncatchable error.
Either unparsed-text() should raise FOUT1190 (in which case unparsed-text-available() would return false), or we should ignore the encoding given in the file and continue as if it wasn't there.
Updated by Michael Kay over 1 year ago
- Status changed from New to In Progress
- Priority changed from Low to Normal
Unfortunately the QT3 test driver uses its own resolver for unparsed-text()
and this masks the error.
Running it from the command line, though, I can reproduce the problem. Almost, anyway: with encoding="foo"
I get java.nio.charset.UnsupportedCharsetException
, and with encoding="fo o"
I get java.nio.charset.IllegalCharsetNameException
.
Updated by Michael Kay over 1 year ago
The standard resolver invokes ResourceLoader.urlReader()
, which returns ResourceLoader.inferEncoding()
, which returns "foo". It then calls getReaderForStream()
, which calls Charset charset2 = Charset.forName(resourceEncoding);
which is where the exception occurs.
The most obvious action here is to catch this exception and throw FOUT1190. However we're on a path here that's used for other things beyond unparsed-text(), so we need to look carefully at the other users of the code to check that this makes sense.
Updated by Michael Kay over 1 year ago
The following appears to work:
(a) Change ResourceLoader.getReaderFromStream()
to catch any unchecked exceptions and throw an UnsupportedEncodingException
.
(b) Change StandardUnparsedTextResolver()
to catch the UnsupportedEncodingException
and turn it into an FOUT1170.
Updated by Michael Kay over 1 year ago
- Status changed from In Progress to Resolved
- Applies to branch 11, 12, trunk added
- Fix Committed on Branch 11, 12, trunk added
- Platforms Java added
Updated by O'Neil Delpratt over 1 year ago
- % Done changed from 0 to 100
- Fixed in Maintenance Release 12.2 added
Bug fix applied in the Saxon 12.2 maintenance release.
Updated by Debbie Lockett about 1 year ago
- Status changed from Resolved to Closed
- Fixed in Maintenance Release 11.6 added
Bug fix applied in the Saxon 11.6 maintenance release.
Please register to edit this issue