Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

appmap.yml packages path matching zero classes yields a healthy-looking recording with no warning

Abierto
  • ¿Algo no está bien?
#332 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
45/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
java
Área
observability

Línea de trabajo

Start with the packages: handling in appmap.yml config loading and the bytecode instrumentation of application classes; the issue does not name the source files. Reproduce by running PetClinicIntegrationTests#testOwnerDetails with a misspelled package path and check the recording size and stderr. Done when a path matching zero classes prints a warning naming that entry, and an all-zero config fails with an error.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Summary

An appmap.yml whose packages: paths match zero classes produces a recording that looks healthy (HTTP route, status, SQL all present) with no warning. Not one line of application code is captured. With no appmap.yml the agent auto-detects the package and the recording is complete, so a wrong config is strictly worse than no config.

Agent 1.30.1, CLI 3.203.0, spring-petclinic @ b26f235, Java 17.

Repro

# appmap.yml  (real package is ...petclinic.owner; one-character typo)
packages:
  - path: org.springframework.samples.petclinic.owners

mvn -Dtest=PetClinicIntegrationTests#testOwnerDetails test
appmap query endpoints --appmap-dir tmp/appmap
appmap query find queries --appmap-dir tmp/appmap --limit 3

Observed

No warning from agent or CLI. endpoints shows GET /owners/{ownerId} 200, find queries shows the eight expected SQL statements. The only tell is the file size (3.2 KB vs 8.6 KB with the right config) and an empty appmap query hotspots.

Cause

packages: governs only bytecode instrumentation of application classes; HTTP and JDBC events come from framework hooks that do not consult it. "Zero classes matched" is not treated as an error or warning, and the configured path is never validated against the classpath.

Expected

At the end of a recording the agent knows how many classes each path: entry instrumented. A path: matching zero classes should warn on stderr naming the entry; a config where every entry matched zero classes should be a hard error, not a silent 3 KB recording.

Written by Devin

Lenguaje dominante
Java
Estrellas
88
Forks
17
Merge medio
2 d 2 h
PR fusionados (30 d)
1

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de getappmap/appmap-java

Todos los issues de getappmap/appmap-java

Issues similares

Más issues de Java

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.