kinow commented on pull request #124: URL: https://github.com/apache/commons-imaging/pull/124#issuecomment-799730283
> By the way, again related to IPTC metadata, if there are more than one **Photoshop App13** segments in a jpeg image, JpegImageParser throws > > > ImageReadException with "Jpeg contains more than one Photoshop App13 segment" message. > > Also, JpegIptcRewriter throws the same exception when writing metadata to a jpeg image containing more than one App13 segments. > > In my local branch, i merged multiple App13 segments into a single one. But those multiple segments can also be added to **JpegPhotoshopMetadata** as an array named **photoshopApp13Datas** instead of **photoshopApp13Data**. > > Do you think something should be done about that? > > Best. https://issues.apache.org/jira/browse/IMAGING-100 feel free to read that issue and suggest a fix @yavuzkavus . There's already a test image for that, so we should be able to write a unit test too. Would be nice if we could find somewhere a specification or some other documentation confirming whether it's valid for the file format to have multiple photoshop segments. If not, we can still add it, but just need to either document in the source code, or use a flag somewhere to allow users to run in a more strict mode. Thanks! Bruno ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
