Remove usages of json-simple

[gbrodman]

We should use gson wherever possible. There's no point in having unnecessary dependencies (we'll need to keep around jackson for YAML parsing).

---

This change is 

[github-actions]

⚠️ Attention Required: Lockfile Detected

This pull request contains modifications to one or more *.lockfile files. Please confirm that you have run update_dependency.sh to push new dependencies to the private repo.

Someone with Admin role must manually dismiss this review before merging.

ran it

[jicelhay]

@jicelhay reviewed 35 files and all commit messages, and made 4 comments.
Reviewable status: 35 of 37 files reviewed, 4 unresolved discussions (waiting on gbrodman).

---

common/gradle.lockfile line 37 at r1 (raw file):

io.github.eisop:dataflow-errorprone:3.41.0-eisop1=annotationProcessor,testAnnotationProcessor,testingAnnotationProcessor
io.github.java-diff-utils:java-diff-utils:4.12=annotationProcessor,testAnnotationProcessor,testingAnnotationProcessor
io.github.java-diff-utils:java-diff-utils:4.16=compileClasspath,deploy_jar,runtimeClasspath,testCompileClasspath,testRuntimeClasspath,testing,testingCompileClasspath

not related ot json, right?

---

console-webapp/package-lock.json line 631 at r1 (raw file):

      }
    },
    "node_modules/@angular/build/node_modules/@types/node": {

same for these dependencies.

---

core/src/main/java/google/registry/request/RequestModule.java line 212 at r1 (raw file):

    }
    try {
      return Preconditions.checkNotNull(gson.fromJson(payload, new TypeToken<>() {}));

nit: import these 2 static methods.

---

core/src/main/java/google/registry/security/JsonHttp.java line 45 at r1 (raw file):

  private static final FluentLogger logger = FluentLogger.forEnclosingClass();

  private static final Gson GSON = GsonUtils.provideGson();

same for these methods

[github-actions]

⚠️ Attention Required: Lockfile Detected

This pull request contains modifications to one or more *.lockfile files. Please confirm that you have run update_dependency.sh to push new dependencies to the private repo.

Someone with Admin role must manually dismiss this review before merging.

i did

[gbrodman]

@gbrodman made 4 comments.
Reviewable status: 20 of 37 files reviewed, 4 unresolved discussions (waiting on jicelhay).

---

core/src/main/java/google/registry/request/RequestModule.java line 212 at r1 (raw file):

Previously, jicelhay (Juan Celhay) wrote…

nit: import these 2 static methods.

which two? why? we have many instances of both "statically import the method" and "use Preconditions" in the codebase

---

core/src/main/java/google/registry/security/JsonHttp.java line 45 at r1 (raw file):

Previously, jicelhay (Juan Celhay) wrote…

same for these methods

no, these should not be statically imported. 1. the qualification distinguishes where we're getting it from 2. we do it this way all throughout the codebase

---

common/gradle.lockfile line 37 at r1 (raw file):

Previously, jicelhay (Juan Celhay) wrote…

not related ot json, right?

yeah these just get updated whenever we run the full build with --write-lockfiles

---

console-webapp/package-lock.json line 631 at r1 (raw file):

Previously, jicelhay (Juan Celhay) wrote…

same for these dependencies.

same