Proper way of hooking methods named `call`
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 35/100
Piste de recherche
Commencez dans lib/appmap/hook.rb, en particulier dans trace_end et la logique qui ignore les méthodes nommées call. Consultez le contexte de la pull request liée pour comprendre pourquoi call est ignoré globalement. Le travail est terminé lorsque les méthodes de l’application nommées call dans app ou lib peuvent être instrumentées sans monkey patch et que les exclusions existantes restent intentionnelles.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
It looks that we globally ignore methods named call here.
When we want to hook methods named call in our application Ruby classes (./app or ./lib in our case), is there any proper way to do this?
Currently, I'm working around it with a monkey patch like this:
# lib/appmap/hook.rb
module AppMap
class Hook
def trace_end(...)
...
local_path = location.delete_prefix(Dir.pwd + '/')
is_app = local_path.start_with?('app') || local_path.start_with?('lib')
next if !is_app && method_id == :call
But I think there must be a better way to do this.
- Langage dominant
- Ruby
- Étoiles
- 101
- Forks
- 13
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
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.
Autres issues de getappmap/appmap-ruby
-
Difficulté 4/5 3-5 jours Accessibilité débutants 30/100
getappmap/appmap-ruby#363 · 2 commentaires ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
getappmap/appmap-ruby#360 ·
-
enhancement
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
getappmap/appmap-ruby#357 ·
-
bug
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
getappmap/appmap-ruby#355 ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
getappmap/appmap-ruby#340 ·
Toutes les issues de getappmap/appmap-ruby
Issues similaires
-
bug
Difficulté 1/5 Moins d'une heure Accessibilité débutants 86/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
rubys/roundhouse#571 ·
Les mainteneurs répondent en général sous 1 jour
-
area/web interface
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
mastodon/mastodon#40924 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
area: cli enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
krzemienski/awesome-video#143 ·