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

/all function throws an error #2

Open
hvdklauw opened this issue Jun 20, 2013 · 5 comments
Open

/all function throws an error #2

hvdklauw opened this issue Jun 20, 2013 · 5 comments

Comments

@hvdklauw
Copy link

It actually outputs it to me:

File "keychain/app.py", line 115, in all_keys
keys_ = [lookup_key(email, key) for key in keys[email]]
NameError: global name 'keys' is not defined

@sysr-q
Copy link

sysr-q commented Jun 20, 2013

Confirmed, I get the same output:

% curl -s ssh.keychain.io/$email/all                           22:24:52
Traceback (most recent call last):
  File "/app/.heroku/python/lib/python2.7/site-packages/eventlet/wsgi.py", line 382, in handle_one_response
    result = self.application(self.environ, start_response)
  File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1701, in __call__
    return self.wsgi_app(environ, start_response)
  File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1689, in wsgi_app
    response = self.make_response(self.handle_exception(e))
  File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1687, in wsgi_app
    response = self.full_dispatch_request()
  File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1360, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1358, in full_dispatch_request
    rv = self.dispatch_request()
  File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1344, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "keychain/app.py", line 115, in all_keys
    keys_ = [lookup_key(email, key) for key in keys[email]]
NameError: global name 'keys' is not defined

cinaglia pushed a commit to cinaglia/keychain.io that referenced this issue Oct 18, 2013
progrium added a commit that referenced this issue Oct 18, 2013
Fixed issue #2: /all errors
@RyanBalfanz
Copy link
Contributor

I just hit this too:

$ curl -s ssh.keychain.io/<email>/all
Traceback (most recent call last):
  File "/app/.heroku/python/lib/python2.7/site-packages/eventlet/wsgi.py", line 382, in handle_one_response
    result = self.application(self.environ, start_response)
  File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1701, in __call__
    return self.wsgi_app(environ, start_response)
  File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1689, in wsgi_app
    response = self.make_response(self.handle_exception(e))
  File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1687, in wsgi_app
    response = self.full_dispatch_request()
  File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1360, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1358, in full_dispatch_request
    rv = self.dispatch_request()
  File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1344, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "keychain/app.py", line 115, in all_keys
    keys_ = [lookup_key(email, key) for key in keys[email]]
NameError: global name 'keys' is not defined

@progrium
Copy link
Owner

Probably a simple fix. I'm sure any of you could go for it.

On Tue, Jan 28, 2014 at 12:14 PM, Ryan Balfanz [email protected]:

I just hit this too:

$ curl -s ssh.keychain.io//all
Traceback (most recent call last):
File "/app/.heroku/python/lib/python2.7/site-packages/eventlet/wsgi.py", line 382, in handle_one_response

result = self.application(self.environ, start_response)
File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1701, in call
return self.wsgi_app(environ, start_response)
File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1689, in wsgi_app
response = self.make_response(self.handle_exception(e))
File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1687, in wsgi_app
response = self.full_dispatch_request()
File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1360, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1358, in full_dispatch_request
rv = self.dispatch_request()
File "/app/.heroku/python/lib/python2.7/site-packages/flask/app.py", line 1344, in dispatch_request
return self.view_functionsrule.endpoint

File "keychain/app.py", line 115, in all_keys
keys_ = [lookup_key(email, key) for key in keys[email]]
NameError: global name 'keys' is not defined

Reply to this email directly or view it on GitHubhttps://github.com//issues/2#issuecomment-33506759
.

Jeff Lindsay
http://progrium.com

@RyanBalfanz
Copy link
Contributor

@progrium, I'm unable to reproduce locally. Perhaps keychain.io isn't running the latest version?

Steps to try to reproduce:

  • Fork
  • Create a new S3 bucket in my AWS account
  • Add my AWS and SendGrid credentials to .env
  • foreman start
  • curl -s http://127.0.0.1:5000/[email protected]/upload | bash
  • Wait for email, click confirmation link -> "Action completed"
  • curl -s http://127.0.0.1:5000/[email protected]/all -> spits out my key as expected

@cinaglia
Copy link
Contributor

cinaglia commented Feb 1, 2014

The fix (cinaglia@a1725cf) for this issue has been merged in a0f7017. I'm guessing it has never been pushed to production, though.

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

No branches or pull requests

5 participants