[Archive-crawler-cvs] [internetarchive/heritrix3] 055f4f: Require Java 17 minimum

[Alex O. <no...@gi...>]

  Branch: refs/heads/master
  Home:   https://github.com/internetarchive/heritrix3
  Commit: 055f4f9f2217e047303e0f82d42ec7311e29c092
      https://github.com/internetarchive/heritrix3/commit/055f4f9f2217e047303e0f82d42ec7311e29c092
  Author: Alex Osborne <aos...@nl...>
  Date:   2024-11-20 (Wed, 20 Nov 2024)

  Changed paths:
    M .github/workflows/maven.yml
    M docs/getting-started.rst
    M pom.xml

  Log Message:
  -----------
  Require Java 17 minimum


  Commit: dd210c8e7014f919269bdf74cd353b474c027b76
      https://github.com/internetarchive/heritrix3/commit/dd210c8e7014f919269bdf74cd353b474c027b76
  Author: Alex Osborne <aos...@nl...>
  Date:   2024-11-20 (Wed, 20 Nov 2024)

  Changed paths:
    M commons/src/main/java/org/archive/checkpointing/Checkpoint.java
    M commons/src/main/java/org/archive/spring/ConfigPath.java
    M commons/src/main/java/org/archive/spring/PathSharingContext.java
    A commons/src/main/java/org/archive/spring/Required.java
    A commons/src/main/java/org/archive/spring/RequiredAnnotationBeanPostProcessor.java
    M commons/src/main/java/org/archive/spring/Sheet.java
    M engine/src/main/java/org/archive/crawler/spring/DecideRuledSheetAssociation.java
    M engine/src/main/java/org/archive/crawler/spring/SurtPrefixesSheetAssociation.java
    M modules/src/main/java/org/archive/modules/ScriptedProcessor.java
    M modules/src/main/java/org/archive/modules/deciderules/ScriptedDecideRule.java
    M modules/src/main/java/org/archive/modules/deciderules/ViaSurtPrefixedDecideRule.java
    M modules/src/main/java/org/archive/modules/seeds/TextSeedModule.java

  Log Message:
  -----------
  Replace Spring's removed @Required annotation with our own

Spring 6 removed @Required and they suggest using constructor injection instead. If we switched our beans to that we'd break existing Heritrix crawl configs. So this change implements our own basic version so we still get errors when a @Required property is null.


  Commit: 205cd2a64b8fb4822e912bfc378888de44aeb076
      https://github.com/internetarchive/heritrix3/commit/205cd2a64b8fb4822e912bfc378888de44aeb076
  Author: Alex Osborne <aos...@nl...>
  Date:   2024-11-20 (Wed, 20 Nov 2024)

  Changed paths:
    M commons/pom.xml

  Log Message:
  -----------
  Bump spring.version from 5.3.39 to 6.1.15


  Commit: dbed84740b959139f42ac585122267f50f9765db
      https://github.com/internetarchive/heritrix3/commit/dbed84740b959139f42ac585122267f50f9765db
  Author: Alex Osborne <aos...@nl...>
  Date:   2024-11-24 (Sun, 24 Nov 2024)

  Changed paths:
    M pom.xml

  Log Message:
  -----------
  Enable the -parameters compiler option

This is needed for Spring 6.1 to be able to reflect on parameter names. We have some autowiring that relies on parameter names such as `AbstractFrontier.setScope(DecideRule scope)`. Without this Spring doesn't know whether to inject the `scope` or `acceptSurts` beans as they both subclass DecideRule. Apparently older versions of Spring used to discover method names by parsing bytecode but this was mechanism removed in 6.1.


  Commit: 6de204c53cbcf13db4431ffdbec919c90c1a359c
      https://github.com/internetarchive/heritrix3/commit/6de204c53cbcf13db4431ffdbec919c90c1a359c
  Author: Alex Osborne <aos...@nl...>
  Date:   2024-11-24 (Sun, 24 Nov 2024)

  Changed paths:
    M commons/src/main/java/org/archive/bdb/AutoKryo.java

  Log Message:
  -----------
  AutoKryo: Replace SerializingInstantiatorStrategy with StdInstantiatorStrategy

SerializingInstantiatorStrategy when compiled for Java 17 fails when running without `--add-opens java.base/java.io=ALL-UNNAMED`. StdInstantiatorStrategy seems to work OK though.


  Commit: a02a33983050365b1abc73fa50fce491808bb5e0
      https://github.com/internetarchive/heritrix3/commit/a02a33983050365b1abc73fa50fce491808bb5e0
  Author: Alex Osborne <aos...@nl...>
  Date:   2024-11-24 (Sun, 24 Nov 2024)

  Changed paths:
    M engine/src/main/java/org/archive/crawler/restlet/JobRelatedResource.java

  Log Message:
  -----------
  Fix StackOverflowError and map exceptions in Browse Beans


  Commit: 8ec614224ebe7ca06e643350354d929ebe6aac41
      https://github.com/internetarchive/heritrix3/commit/8ec614224ebe7ca06e643350354d929ebe6aac41
  Author: Alex Osborne <aos...@nl...>
  Date:   2024-11-28 (Thu, 28 Nov 2024)

  Changed paths:
    M .github/workflows/maven.yml
    M commons/pom.xml
    M commons/src/main/java/org/archive/bdb/AutoKryo.java
    M commons/src/main/java/org/archive/checkpointing/Checkpoint.java
    M commons/src/main/java/org/archive/spring/ConfigPath.java
    M commons/src/main/java/org/archive/spring/PathSharingContext.java
    A commons/src/main/java/org/archive/spring/Required.java
    A commons/src/main/java/org/archive/spring/RequiredAnnotationBeanPostProcessor.java
    M commons/src/main/java/org/archive/spring/Sheet.java
    M docs/getting-started.rst
    M engine/src/main/java/org/archive/crawler/restlet/JobRelatedResource.java
    M engine/src/main/java/org/archive/crawler/spring/DecideRuledSheetAssociation.java
    M engine/src/main/java/org/archive/crawler/spring/SurtPrefixesSheetAssociation.java
    M modules/src/main/java/org/archive/modules/ScriptedProcessor.java
    M modules/src/main/java/org/archive/modules/deciderules/ScriptedDecideRule.java
    M modules/src/main/java/org/archive/modules/deciderules/ViaSurtPrefixedDecideRule.java
    M modules/src/main/java/org/archive/modules/seeds/TextSeedModule.java
    M pom.xml

  Log Message:
  -----------
  Merge pull request #625 from internetarchive/spring-6.1

Upgrade to Spring 6.1 (and Java 17 minimum)


Compare: https://github.com/internetarchive/heritrix3/compare/13075ec4442c...8ec614224ebe

To unsubscribe from these emails, change your notification settings at https://github.com/internetarchive/heritrix3/settings/notifications