GooDer commented on PR #194:
URL: https://github.com/apache/pdfbox/pull/194#issuecomment-2119747971

   Hi @THausherr sorry for late reply.
   
   I agree that PDF is not in good shape, but unfortunately it is some input 
from users / companies which we need to process even if there are some problems.
   
   Logging that error is problem for our alerting system, which is triggered 
for every error logged.
   
   I can see three possibilities, how to handle this:
   
   1. as proposed in PR, change log level to warning. PDF is still processed 
even that this error occure
   2. throw exception instead of logging error so user of the PDFBox have to 
react on it and decide what to do
   3. switch of error logging for that class, which could be dangerous
   
   Please suggest what would be the best choice in this case.
   
   Thanks you


-- 
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...@pdfbox.apache.org

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


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

Reply via email to