jbonofre commented on PR #2980: URL: https://github.com/apache/avro/pull/2980#issuecomment-2694374226
@podmak that's a good question. What I proposed was an implicit loading in `ReflectData` without impact on existing code, so without changing the constructor. That's why I did the `static` init and check in the `SpecificDatumReader(writer, reader, data)` constructor, as it's only one I found used. I think you are right, `SpecificDatumReader(data)` constructor should also check the trusted package. Let me do a new PR to fix that. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
