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

Exception when no auth configured #8

Closed
zawwz opened this issue Feb 25, 2022 · 1 comment
Closed

Exception when no auth configured #8

zawwz opened this issue Feb 25, 2022 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@zawwz
Copy link

zawwz commented Feb 25, 2022

There is an exception when you try to access the page without configuring auth/user

Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 2073, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1518, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1516, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1502, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**req.view_args)
  File "/app/homed.py", line 25, in display_home
    app.logger.info("user={user}, path=/".format(user=user["username"]))
KeyError: 'username'

Very cool project otherwise, keep up the good work!

@zawwz zawwz added the bug Something isn't working label Feb 25, 2022
@mwalters mwalters self-assigned this Mar 27, 2022
mwalters added a commit that referenced this issue Mar 27, 2022
Remediate #8 by supplying an anonymous user in cases where a user system is not in use
@mwalters mwalters mentioned this issue Mar 27, 2022
mwalters added a commit that referenced this issue Mar 27, 2022
Remediate #8 by supplying an anonymous user in cases where a user system is not in use
@mwalters
Copy link
Owner

Sorry, missed this report. Thanks for letting me know @zawwz -- #9 should fix this (included in v1.3.1)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants