[ https://issues.apache.org/jira/browse/PIRK-45?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15558296#comment-15558296 ]
ASF GitHub Bot commented on PIRK-45: ------------------------------------ Github user ellisonanne commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/107#discussion_r82505758 --- Diff: src/main/java/org/apache/pirk/query/wideskies/Query.java --- @@ -26,100 +26,103 @@ import java.util.concurrent.ConcurrentHashMap; import java.util.function.Consumer; +import com.fasterxml.jackson.annotation.JsonIgnore; --- End diff -- Need to remove Jackson imports > Remove dependency on Java serialization > --------------------------------------- > > Key: PIRK-45 > URL: https://issues.apache.org/jira/browse/PIRK-45 > Project: PIRK > Issue Type: Improvement > Reporter: Tim Ellison > Assignee: Walter Ray-Dulany > > Pirk should not depend upon Java serialization as a persistent object format. > Maintaining support for a variety of versions of Java serialized form can be > difficult, this includes both the querier and responder sides of a PIR. > Alternative formats such as XML and JSON are more forgiving/extensible. > Furthermore, and despite Pirk's trust between querier and responder, there > are potential implications for loading the binary representation of Java's > serialized instances as a vector for security vulnerabilities. -- This message was sent by Atlassian JIRA (v6.3.4#6332)