Skip to content

Commit

Permalink
TIKA-4221 -- temporary workaround for COMPRESS-675 (#1686) -- fix mer…
Browse files Browse the repository at this point in the history
…ge conflicts (#1750)
  • Loading branch information
tballison authored May 8, 2024
1 parent ebdb6fc commit 84d05ad
Showing 1 changed file with 57 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@
import org.apache.commons.compress.compressors.snappy.SnappyCompressorInputStream;
import org.apache.commons.compress.compressors.xz.XZCompressorInputStream;
import org.apache.commons.compress.compressors.z.ZCompressorInputStream;
import org.apache.commons.io.input.CloseShieldInputStream;
import org.apache.commons.io.input.ClosedInputStream;
import org.xml.sax.ContentHandler;
import org.xml.sax.SAXException;

Expand Down Expand Up @@ -170,10 +170,10 @@ public void parse(InputStream stream, ContentHandler handler, Metadata metadata,
// any associated resources, but the underlying document stream
// should not be closed
if (stream.markSupported()) {
stream = CloseShieldInputStream.wrap(stream);
stream = new TikaCloseShieldInputStream(stream);
} else {
// Ensure that the stream supports the mark feature
stream = new BufferedInputStream(CloseShieldInputStream.wrap(stream));
stream = new BufferedInputStream(new TikaCloseShieldInputStream(stream));
}

CompressorInputStream cis;
Expand Down Expand Up @@ -267,4 +267,58 @@ public boolean isDecompressConcatenated() {
return this.decompressConcatenated;
}

//TODO -- get rid of this workaround once TIKA-4221/COMPRESS-675 are fixed
private class TikaCloseShieldInputStream extends InputStream {

private InputStream stream;
public TikaCloseShieldInputStream(InputStream stream) {
this.stream = stream;
}

@Override
public void close() throws IOException {
this.stream = ClosedInputStream.INSTANCE;
}


@Override
public int read(byte[] b) throws IOException {
return stream.read(b);
}

@Override
public int read(byte[] b, int off, int len) throws IOException {
return stream.read(b, off, len);
}

@Override
public long skip(long n) throws IOException {
return stream.skip(n);
}

@Override
public int available() throws IOException {
return stream.available();
}

@Override
public synchronized void mark(int readlimit) {
stream.mark(readlimit);
}

@Override
public synchronized void reset() throws IOException {
stream.reset();
}

@Override
public boolean markSupported() {
return stream.markSupported();
}

@Override
public int read() throws IOException {
return stream.read();
}
}
}

0 comments on commit 84d05ad

Please sign in to comment.