[OpenNMS/opennms] d00c66: NMS-20163: Redirect api-docs to api-docs/

  Branch: refs/heads/mm/NMS-20163-smoke
  Home:   https://github.com/OpenNMS/opennms
  Commit: d00c66ed2be7859443a2060236aafee6e55e4506
      https://github.com/OpenNMS/opennms/commit/d00c66ed2be7859443a2060236aafee6e55e4506
  Author: Marshall Massengill <[email protected]>
  Date:   2026-08-11 (Tue, 11 Aug 2026)

  Changed paths:
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/support/openapi/AbstractSwaggerUiResource.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/support/openapi/SwaggerUiResourceTest.java

  Log Message:
  -----------
  NMS-20163: Redirect api-docs to api-docs/

index.html is served at both spellings, but it names its assets and its
spec URL relative to itself, so from the slash-less one they all resolve
a directory too high and the page comes up broken. Every link we ship
carries the slash, so this is only reachable by typing the URL.


  Commit: d0fa522a44a2f926f5bd88455a885568d82cad35
      https://github.com/OpenNMS/opennms/commit/d0fa522a44a2f926f5bd88455a885568d82cad35
  Author: Marshall Massengill <[email protected]>
  Date:   2026-08-11 (Tue, 11 Aug 2026)

  Changed paths:
    M opennms-openapi-docs/pom.xml

  Log Message:
  -----------
  NMS-20163: Keep the generator off the Windows command line limit

The forked generator was handed the whole project classpath inline after
-classpath, which on this project runs past the 32K that CreateProcess
accepts, so the build failed on Windows before the JVM started.
longClasspath moves the classpath into the manifest of a generated jar;
the documents come out byte-identical, and the fork check still holds
because the pathing jar is still loaded by the system class loader.


  Commit: cd561a90c74069ca88fa0fc5eff35eaeb888eb48
      https://github.com/OpenNMS/opennms/commit/cd561a90c74069ca88fa0fc5eff35eaeb888eb48
  Author: Marshall Massengill <[email protected]>
  Date:   2026-08-11 (Tue, 11 Aug 2026)

  Changed paths:
    A smoke-test/src/test/java/org/opennms/smoketest/rest/OpenApiRestIT.java

  Log Message:
  -----------
  NMS-20163: Cover the OpenAPI endpoints with a smoke test

The documents and the Swagger UI are read off the classpath, and the
resources serving them answer 404 rather than failing when they are
absent. Nothing short of a request against a running instance notices a
packaging slip, such as the documents jar dropping out of webapp-full.
This replaces the OpenApiIT that generation at build time made obsolete,
which ran in-process against a classpath that no longer carries the
documents.


Compare: https://github.com/OpenNMS/opennms/compare/bb0f2a000dc1...cd561a90c740

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


_______________________________________________
Please read the OpenNMS Mailing List FAQ:
http://www.opennms.org/wiki/index.php?page=MailingListFaq
opennms-cvs mailing list

To *unsubscribe* or change your subscription options, see the bottom of this page:
https://lists.sourceforge.net/lists/listinfo/opennms-cvs