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

NoMethodError: undefined method `[]' for nil:NilClass #190

Closed
Tracked by #188
nevans opened this issue Oct 3, 2023 · 0 comments · Fixed by #192
Closed
Tracked by #188

NoMethodError: undefined method `[]' for nil:NilClass #190

nevans opened this issue Oct 3, 2023 · 0 comments · Fixed by #192

Comments

@nevans
Copy link
Collaborator

nevans commented Oct 3, 2023

/bundler/gems/net-imap-e76475223002/lib/net/imap.rb:2632 in block in search_internal
/usr/local/lib/ruby/3.1.0/monitor.rb:202 in synchronize
/usr/local/lib/ruby/3.1.0/monitor.rb:202 in mon_synchronize
/bundler/gems/net-imap-e76475223002/lib/net/imap.rb:2626 in search_internal
/bundler/gems/net-imap-e76475223002/lib/net/imap.rb:1850 in search
lib/redacted.rb:138 in search_since
  137:  def search_since(date)
  138:    imap_client.search("SINCE #{date.strftime("%d-%b-%Y")}")
  139:  end
@nevans nevans mentioned this issue Oct 3, 2023
7 tasks
@nevans nevans changed the title undefined method `[]' for nil:NilClass ResponseParseError: undefined method `[]' for nil:NilClass Oct 3, 2023
@nevans nevans changed the title ResponseParseError: undefined method `[]' for nil:NilClass NoMethodError: undefined method `[]' for nil:NilClass Oct 3, 2023
nevans added a commit that referenced this issue Oct 4, 2023
By converting all of the internal `@responses.delete(type)` to
`clear_responses(type)`, we get empty arrays and don't need nil-checks.

Fixes #190.
nevans added a commit that referenced this issue Oct 4, 2023
By converting all of the internal `@responses.delete(type)` to
`clear_responses(type)`, we get empty arrays and don't need nil-checks.

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

Successfully merging a pull request may close this issue.

1 participant