getappmap / getappmap/appmap-java
Normalized paths not present in AppMaps with shallow-recorded spring framework classes
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Java
- Sterne
- 88
- Forks
- 17
- Ø Merge
- 2 T. 2 Std.
- Gemergte PRs (30 T.)
- 1
Beschreibung
For the same app & appmap.jar, the HTTP server request events are different:
without spring framework events:
"event": "call",
"http_server_request": {
"normalized_path_info": "/owners/:ownerId",
"path_info": "/owners/2",
"protocol": "HTTP/1.1",
"request_method": "GET"
},
with spring framework events:
"event": "call",
"http_server_request": {
"path_info": "/owners/2",
"protocol": "HTTP/1.1",
"request_method": "GET"
},
appmap.yml:
name: spring-petclinic
# Your project contains the directory src/main/java.
# AppMap has auto-detected the following Java packages in this directory:
packages:
- path: org.springframework.samples.petclinic
- path: org.springframework.web
shallow: true
exclude:
- org.springframework.web.filter.OncePerRequestFilter
FindOwnerDavis.appmap.json.zip
Spring_Find_Owner.appmap.json.zip
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginnen Sie mit den beiden angehängten AppMap JSON-Archiven und der Paketkonfiguration appmap.yml. Vergleichen Sie die Generierung von HTTP-Server-Anfrageereignissen mit und ohne shallow-recorded Spring-Klassen und konzentrieren Sie sich darauf, warum normalized_path_info ausgelassen wird. Als erledigt gilt die Aufgabe, wenn normalisierte Pfade in den betroffenen AppMaps vorhanden bleiben, sobald Spring-Framework-Klassen shallow-recorded werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java, spring
- Bereich
- api, backend
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 38/100