microsoft / microsoft/java-debug
runInTerminal reverse-request missing cwd key
Personne n'a encore pris cette issue.
- Langage dominant
- Java
- Étoiles
- 409
- Forks
- 204
- Merge moyen
- 1 j 13 h
- PR mergées (30 j)
- 4
Description
According to the DAP specification, the cwd key in the runInTerminal request is mandatory.
https://microsoft.github.io/debug-adapter-protocol/specification#Reverse_Requests_RunInTerminal
however, the java debug server sends the request without this key:
2018-12-20 00:03:04,861 - DEBUG - Message received: {'command': 'runInTerminal', 'arguments': {'kind': 'integrated', 'title': 'Java Debug Console', 'args': ['/Library/Java/JavaVirtualMachines/jdk1.8.0_162.jdk/Contents/Home/jre/bin/java', '-agentlib:jdwp=transport=dt_socket,server=n,suspend=y,address=localhost:53159', '-Dfile.encoding=UTF-8', '-cp', '/Users/ben/.vim/bundle/vimspector/support/test/java/test_project/target/classes', 'com.vimspector.test.TestApplication', 'hello', 'world!']}, 'seq': 2, 'type': 'request'}
of course, clients could work around this, but it would be nice for the server to include the required parameter.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par localiser le code du serveur de débogage Java qui construit la requête inverse DAP runInTerminal, puis comparez ses arguments avec la spécification DAP liée. Le travail est terminé lorsque la requête inclut la clé obligatoire cwd et que le comportement pertinent de la requête est couvert par les tests du projet.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- java
- Domaine
- devtools
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 45/100