swagnerfaw opened a new pull request, #396:
URL: https://github.com/apache/poi/pull/396

   MutiValued Property Types inside `__properties_version1.0` caused the 
reading of the properties stream to stop and skip all the rest of the stream 
(missing out on alot of properties in some cases).
   CLS ID GUID has a fixed length, however according to 
[](https://learn.microsoft.com/en-us/openspecs/exchange_server_protocols/ms-oxmsg/08185828-e9e9-4ef2-bcd2-f6e69c00891b)
 it is not read in the `__properties_version1.0`  stream. It is (like variable 
length) properties in another file. So I added another method `isPointer` since 
no property longer than 8 bytes should be included inside the 
`__properties_version1.0`.
   
   I included my test-message here. 
   [Test1.zip](https://github.com/apache/poi/files/9932914/Test1.zip)
   
   ```java
   MAPIMessage msg = new MAPIMessage("Test1.msg");
   System.out.println("#found properties: " + 
msg.getMainChunks().getProperties().size());
   ```
   
   The current 5.2.3 extracts 25 properties
   With my patch all 151 properties included in  `__properties_version1.0`
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

Reply via email to