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 #135

Closed
raucao opened this issue Jan 8, 2020 · 2 comments
Closed

NoMethodError: undefined method `/' for nil:NilClass #135

raucao opened this issue Jan 8, 2020 · 2 comments
Labels

Comments

@raucao
Copy link
Member

raucao commented Jan 8, 2020

https://sentry.io/organizations/5apps/issues/1426307576/?referrer=github_plugin

NoMethodError: undefined method `/' for nil:NilClass
  lib/remote_storage/rest_provider.rb:503:in `block in get_directory_listing_from_redis_via_lua'
    v["Last-Modified"] = Time.at(v["Last-Modified"]/1000).httpdate
  lib/remote_storage/rest_provider.rb:502:in `each'
    items.reject{|k, _| k.end_with? "/"}.each do |_, v|
  lib/remote_storage/rest_provider.rb:502:in `get_directory_listing_from_redis_via_lua'
    items.reject{|k, _| k.end_with? "/"}.each do |_, v|
  lib/remote_storage/rest_provider.rb:102:in `get_directory_listing'
    items = get_directory_listing_from_redis_via_lua(user, directory)
...
(38 additional frame(s) were not displayed)
@raucao raucao added the bug label Jan 8, 2020
@raucao
Copy link
Member Author

raucao commented Jan 8, 2020

Continuation of #128.

I can reproduce this on staging now. But we don't have the old events anymore to see if it's just the staging Redis DB that may be inconsistent.

@raucao
Copy link
Member Author

raucao commented Jan 11, 2020

Closing again, after we fixed the faulty data. There's nothing to fix about the code here.

@raucao raucao closed this as completed Jan 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant