Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix infinite loop problem with the NameErrors in block exposures #355

Merged
merged 2 commits into from
Aug 26, 2021

Conversation

meinac
Copy link
Contributor

@meinac meinac commented Aug 25, 2021

Calling NameError#message causes the did_you_mean library to endup
being in an infinite loop(Probabaly raises SystemStackError at some
point though).

Related to ruby/did_you_mean#158

Calling `NameError#message` causes the `did_you_mean` library to endup
being in an infinite loop(Probabaly raises `SystemStackError` at some
point though).

Related to ruby/did_you_mean#158
@stanhu
Copy link

stanhu commented Aug 26, 2021

@dblock Would you mind taking a look at this?

@LeFnord
Copy link
Member

LeFnord commented Aug 26, 2021

@meinac … which ruby version cause this bad behaviour?

@Quintasan
Copy link

@LeFnord Currently we're running 2.7.2 and will update to 2.7.4 soon.

@LeFnord
Copy link
Member

LeFnord commented Aug 26, 2021

ah ok, want to ensure it fails on a supported version 🙂

please, can you fix the rubocop stuff – yeap, you don't introduced it … rubocop -a, should do it – thanks

@meinac
Copy link
Contributor Author

meinac commented Aug 26, 2021

@LeFnord I've addressed the Rubocop failure by 9f58f29.

BTW, this is not a conflict with the did_you_mean library but an issue with the way Ruby NameError#message works. For more information, you can check ruby/did_you_mean#158 (comment).

Thanks in advance for any help you can offer on this.

Also, thanks @Quintasan for your quick answer.

@LeFnord
Copy link
Member

LeFnord commented Aug 26, 2021

thanks @meinac for fixing it

@LeFnord LeFnord merged commit 1ddf61d into ruby-grape:master Aug 26, 2021
@meinac meinac deleted the fix_infinite_loop_on_name_errors branch September 8, 2021 15:39
@meinac
Copy link
Contributor Author

meinac commented Sep 8, 2021

Hi @LeFnord, do you have a plan to release a new version containing this fix?

@LeFnord
Copy link
Member

LeFnord commented Sep 15, 2021

yeap, today, see #356

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants