-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
favicon.ico request now causes an Exception #189
Comments
ah, right, of course. yeah, let's add that back |
sbvhev
added a commit
to sbvhev/dash
that referenced
this issue
Mar 12, 2018
denis554
added a commit
to denis554/dash
that referenced
this issue
Apr 9, 2019
HammadTheOne
pushed a commit
to HammadTheOne/dash
that referenced
this issue
May 28, 2021
…ted-git-info-2.8.9 Bump hosted-git-info from 2.7.1 to 2.8.9
HammadTheOne
pushed a commit
that referenced
this issue
Jul 23, 2021
…t-info-2.8.9 Bump hosted-git-info from 2.7.1 to 2.8.9
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It looks like this commit #181 and specifically this line which removed *args and **kwargs from Dash's index function signature is now causing an exception on the initial page load of an app due to the
/facivon.ico
request being handled by the index method, thereby getting apath=favicon.ico
parameter.As mentioned in #152 this is caused by Dash serving the index route as a catchall for otherwise unmatched routes.
The text was updated successfully, but these errors were encountered: