Frozen/Deadlocked Ruby thread instead of NoMethodError
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 42/100
Piste de recherche
Start by reproducing the provided Grape::Entity Invoice example with the two-argument expose block and the missing object receiver. Trace the expose block's yield and error handling to identify why the mistake deadlocks instead of raising NoMethodError. Done means the example raises NoMethodError and a regression test covers the behavior.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
I am not sure if this is a known issue or not, but when yielding to a block w/ 2 arguments (e.g. arity = 2) its possible to freeze your app / test code. The following coding mistake should have resulted in a NoMethodError. Instead, Ruby freezes and I have to kill -9 the thread.
The following will freeze my app in its tracks. It should throw a NoMethodError.
module Entities
class Invoice < Grape::Entity
expose :id
expose(:foo) do |object, options|
bar.to_s
end
end
end
The only mistake above was not referencing object first.
module Entities
class Invoice < Grape::Entity
expose :id
expose(:foo) do |object, options|
object.bar.to_s
end
end
end
I'm happy to try to fix this - but first I'm asking if anyone else has noticed this or maybe there is already a PR for something similar?
Versions
Ruby 2.7.3
Rails 6.1
grape (1.5.3)
grape-entity (0.9.0)
- Langage dominant
- Ruby
- Étoiles
- 730
- Forks
- 155
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni 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 ruby-grape/grape-entity
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
ruby-grape/grape-entity#386 · 7 commentaires · 1 réaction ·
-
Custom Entity options supportOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
ruby-grape/grape-entity#382 ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
ruby-grape/grape-entity#380 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
ruby-grape/grape-entity#373 · 2 commentaires ·
-
Truncate FormatterOuverte
Difficulté 1/5 1-3 heures Accessibilité débutants 45/100
ruby-grape/grape-entity#372 ·
Toutes les issues de ruby-grape/grape-entity
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Homebrew/homebrew-cask#293134 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
notch8/iiif_print#430 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
Les mainteneurs répondent en général sous 1 jour
-
ActionText::Content#to_markdown: <br> in a table cell and empty <tr> produce malformed Markdown tablesPeut-être pris @azmi2409 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
Les mainteneurs répondent en général sous 2 jours