SuccessChanges

Summary

  1. Rename variables Fortify think are hardcoded encryption keys (details)
  2. Move HTTP remotes password encryption where its used (details)
  3. Fix fortify privacy violation concern on RepositoryResolver (details)
  4. Remove unused imports (details)
  5. Clean up code: remove blocks of commented out code (details)
  6. Move command hook scripting to experimental module (details)
Commit c4938caff3bd28144d365a77589af4997187d917 by groldan
Rename variables Fortify think are hardcoded encryption keys
Signed-off-by: Gabriel Roldan <groldan@boundlessgeo.com>
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/RevertOp.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/SquashOp.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/CommitOp.java
Commit aa3a721e8bdd7782d8981f95e65e3f6d23a5f5c1 by groldan
Move HTTP remotes password encryption where its used
Remote is a value object and should have no logic. Move password
encryption/decryption to HttpRemoveResolver where is belongs
Signed-off-by: Gabriel Roldan <groldan@boundlessgeo.com>
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/repository/Remote.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/remotes/RemoteAddOp.java
The file was addedsrc/web/api/src/main/java/org/locationtech/geogig/remote/http/HttpRemotePasswordHook.java
The file was modifiedsrc/web/api/src/main/java/org/locationtech/geogig/remote/http/HttpRemoteResolver.java
The file was addedsrc/web/api/src/main/resources/META-INF/services/org.locationtech.geogig.hooks.CommandHook
The file was modifiedsrc/web/api/src/test/java/org/locationtech/geogig/web/api/commands/RemoteManagementTest.java
The file was modifiedsrc/api/src/test/java/org/locationtech/geogig/repository/RemoteTest.java
Commit 32aa8d5e170372a8740a333fcd52f66af07f8e28 by groldan
Fix fortify privacy violation concern on RepositoryResolver
Signed-off-by: Gabriel Roldan <groldan@boundlessgeo.com>
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/repository/RepositoryResolver.java
The file was modifiedsrc/storage/postgres/src/main/java/org/locationtech/geogig/storage/postgresql/PGRepositoryResolver.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/repository/impl/FileRepositoryResolver.java
The file was modifiedsrc/storage/postgres/src/main/java/org/locationtech/geogig/storage/postgresql/config/ConnectionConfig.java
The file was modifiedsrc/storage/postgres/src/main/java/org/locationtech/geogig/storage/postgresql/config/EnvironmentBuilder.java
Commit e785271493f24f8bdb1b1407ff932ab1708b7461 by groldan
Remove unused imports
Signed-off-by: Gabriel Roldan <groldan@boundlessgeo.com>
The file was modifiedsrc/api/src/test/java/org/locationtech/geogig/model/HashObjectFunnelsTest.java
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/model/RevObjects.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/remotes/RemoteRemoveOp.java
The file was modifiedsrc/storage/cache/caffeine/src/test/java/org/locationtech/geogig/cache/performance/ObjectCacheStressTest.java
The file was modifiedsrc/datastore/src/main/java/org/locationtech/geogig/geotools/data/reader/SimplifyingGeometryReplacer.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/DiffFeature.java
The file was modifiedsrc/datastore/src/test/java/org/locationtech/geogig/geotools/data/GeoGigDiffFeatureSourceTest.java
The file was modifiedsrc/datastore/src/main/java/org/locationtech/geogig/geotools/data/reader/ScreenMapGeometryReplacer.java
Commit 022a2a680178176c0b90db7fd00afd8df733a25a by groldan
Clean up code: remove blocks of commented out code
Signed-off-by: Gabriel Roldan <groldan@boundlessgeo.com>
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/model/internal/ClusteringStrategy.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/DiffFeature.java
The file was modifiedsrc/cli/core/src/main/java/org/locationtech/geogig/cli/GeogigCLI.java
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/repository/Remote.java
Commit a913fbbecbdff51b1357459d18ba739696875273 by groldan
Move command hook scripting to experimental module
The file was addedsrc/experimental/scripting/README.md
The file was removedsrc/core/src/main/java/org/locationtech/geogig/hooks/ShellScriptHook.java
The file was modifiedsrc/experimental/pom.xml
The file was removedsrc/core/src/main/java/org/locationtech/geogig/hooks/JVMScriptHook.java
The file was removedsrc/core/src/main/java/org/locationtech/geogig/hooks/Hookables.java
The file was addedsrc/experimental/scripting/src/main/resources/META-INF/services/org.locationtech.geogig.hooks.CommandHook
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/hooks/CommandHookChain.java
The file was addedsrc/experimental/scripting/src/test/java/org/locationtech/geogig/scripting/GeogigAPITest.java
The file was addedsrc/experimental/scripting/src/test/java/org/locationtech/geogig/scripting/HooksTest.java
The file was addedsrc/experimental/scripting/src/main/java/org/locationtech/geogig/scripting/InitializeHooksHook.java
The file was addedsrc/experimental/scripting/src/test/resources/features/hooks/Hooks.feature
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/repository/impl/FileRepositoryResolver.java
The file was addedsrc/experimental/scripting/src/main/java/org/locationtech/geogig/scripting/ShellScriptHook.java
The file was removedsrc/core/src/test/java/org/locationtech/geogig/test/integration/GeogigAPITest.java
The file was removedsrc/core/src/main/resources/META-INF/services/org.locationtech.geogig.repository.RepositoryResolver
The file was addedsrc/experimental/scripting/src/main/java/org/locationtech/geogig/scripting/JVMScriptHook.java
The file was removedsrc/core/src/main/java/org/locationtech/geogig/hooks/Scripting.java
The file was removedsrc/cli/core/src/test/java/org/locationtech/geogig/cli/test/functional/RunHooksFunctionalTest.java
The file was addedsrc/experimental/scripting/src/test/java/org/locationtech/geogig/scripting/functional/RunHooksFunctionalTest.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/hooks/CommandHook.java
The file was addedsrc/experimental/scripting/src/test/resources/META-INF/services/org.locationtech.geogig.hooks.CommandHook
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/hooks/CommandHooksDecorator.java
The file was removedsrc/cli/core/src/test/resources/features/hooks/Hooks.feature
The file was removedsrc/core/src/main/java/org/locationtech/geogig/hooks/GeoGigAPI.java
The file was addedsrc/experimental/scripting/src/main/java/org/locationtech/geogig/scripting/Scripting.java
The file was modifiedsrc/cli/app/pom.xml
The file was addedsrc/experimental/scripting/pom.xml
The file was removedsrc/core/src/main/resources/org/locationtech/geogig/hooks/pre_commit.js.sample
The file was addedsrc/experimental/scripting/src/main/java/org/locationtech/geogig/scripting/GeoGigAPI.java
The file was addedsrc/experimental/scripting/src/main/resources/org/locationtech/geogig/scripting/pre_commit.js.sample
The file was addedsrc/experimental/scripting/src/main/java/org/locationtech/geogig/scripting/ScriptHookWrapper.java
The file was removedsrc/core/src/test/resources/META-INF/services/org.locationtech.geogig.hooks.CommandHook
The file was removedsrc/core/src/test/java/org/locationtech/geogig/test/integration/HooksTest.java