guessContentType in UploadFileSpec.java only guessed gif/jpg/png/html files. Uploading Tiff (from scanners) and PDF is pretty normal, so I added those.
Added tiff/pdf contenttype for guessing
Logged In: YES user_id=1220573 Originator: NO
The change is now in the subversion repository and will be in the next release.
Thanks I've added a new test case and your patch
/** * test the file content type for several file types e.g. a gif image * modified for patch 1415415 * @throws Exception */ public void testFileContentType() throws Exception { File file = createFile("temp.gif", new byte[] { 1, 2, 3, 4, 0x7f, 0x23 } ); doTestFileContentType(file,null,"image/gif:message.name=temp.gif&message.lines=1"); file = createFile("1x1.tif", new byte[] { (byte)0x03,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x01,(byte)0x01,(byte)0x03,(byte)0x00,(byte)0x01,(byte)0x00, (byte)0x00,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x02,(byte)0x01,(byte)0x03,(byte)0x00,(byte)0x03,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x8A,(byte)0x00, (byte)0x00,(byte)0x00,(byte)0x03,(byte)0x01,(byte)0x03,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x06,(byte)0x01, (byte)0x03,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x02,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x11,(byte)0x01,(byte)0x04,(byte)0x00,(byte)0x01,(byte)0x00, (byte)0x00,(byte)0x00,(byte)0x08,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x15,(byte)0x01,(byte)0x03,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x03,(byte)0x00, (byte)0x00,(byte)0x00,(byte)0x16,(byte)0x01,(byte)0x04,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x17,(byte)0x01, (byte)0x04,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x03,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x1C,(byte)0x01,(byte)0x03,(byte)0x00,(byte)0x01,(byte)0x00, (byte)0x00,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x08,(byte)0x00,(byte)0x08,(byte)0x00,(byte)0x08}); doTestFileContentType(file,null,"image/tiff:message.name=1x1.tif&message.lines=1"); }
Log in to post a comment.
Added tiff/pdf contenttype for guessing
Logged In: YES
user_id=1220573
Originator: NO
The change is now in the subversion repository and will be in the next release.
Logged In: YES
user_id=1220573
Originator: NO
Thanks I've added a new test case and your patch
/**
* test the file content type for several file types e.g. a gif image
* modified for patch 1415415
* @throws Exception
*/
public void testFileContentType() throws Exception {
File file = createFile("temp.gif", new byte[] { 1, 2, 3, 4, 0x7f, 0x23 } );
doTestFileContentType(file,null,"image/gif:message.name=temp.gif&message.lines=1");
file = createFile("1x1.tif", new byte[] {
(byte)0x03,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x01,(byte)0x01,(byte)0x03,(byte)0x00,(byte)0x01,(byte)0x00,
(byte)0x00,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x02,(byte)0x01,(byte)0x03,(byte)0x00,(byte)0x03,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x8A,(byte)0x00,
(byte)0x00,(byte)0x00,(byte)0x03,(byte)0x01,(byte)0x03,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x06,(byte)0x01,
(byte)0x03,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x02,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x11,(byte)0x01,(byte)0x04,(byte)0x00,(byte)0x01,(byte)0x00,
(byte)0x00,(byte)0x00,(byte)0x08,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x15,(byte)0x01,(byte)0x03,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x03,(byte)0x00,
(byte)0x00,(byte)0x00,(byte)0x16,(byte)0x01,(byte)0x04,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x17,(byte)0x01,
(byte)0x04,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x03,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x1C,(byte)0x01,(byte)0x03,(byte)0x00,(byte)0x01,(byte)0x00,
(byte)0x00,(byte)0x00,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x08,(byte)0x00,(byte)0x08,(byte)0x00,(byte)0x08});
doTestFileContentType(file,null,"image/tiff:message.name=1x1.tif&message.lines=1");
}