Review new DangerousJavaDeserialization error-prone check

[findepi]

DangerousJavaDeserialization is a recently added error-prone check (#10788). Figure out whether we adjust the code or suppress the check permanently

[spotted-cat]

Can I work on this issue?

[Fokko]

@spotted-cat sure thing! Feel free to reach out when you bump into anything

[spotted-cat]

@spotted-cat sure thing! Feel free to reach out when you bump into anything

Thankyou for offering to help.

So I looked up into the issue, and I am unable to find any documentation for the check DangerousJavaDeserialization which atleast for now I didn't see listed in either Google's error prone docs or of the Palantir Baseline Error prone checks.

Can you point me if I am missing something here?

[ebyhr]

@spotted-cat You can find the code at https://github.com/palantir/gradle-baseline/blob/develop/baseline-error-prone/src/main/java/com/palantir/baseline/errorprone/DangerousJavaDeserialization.java

[aadvik93]

Hi, I'd like to work on this issue. Could you please assign it to me?

[RussellSpitzer]

We generally do not assign issues since we get a pretty low follow through. That said please let us know when you have a PR ready

[aadvik93]

We generally do not assign issues since we get a pretty low follow through. That said please let us know when you have a PR ready
Got it, thanks! I'll let you know once the PR is ready.