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.

Sign up for GitHub

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

Jump to bottom

启动报错 #2985

Open
1 task done
f754699 opened this issue Jul 13, 2024 · 2 comments · May be fixed by #2992
Open
1 task done

启动报错 #2985

f754699 opened this issue Jul 13, 2024 · 2 comments · May be fixed by #2992
Labels
bug

Comments

@f754699
Copy link

f754699 commented Jul 13, 2024 •
edited by ChihweiLHBird
Loading

Is there an existing issue for this?

  • I have searched the existing issues

Describe the bug

 File "F:\pythonproject\sanicResolve2\.venv\Lib\site-packages\sanic\mixins\startup.py", line 1180, in serve
    while cls._get_process_states(worker_state):
                                  ^^^^^^^^^^^^
UnboundLocalError: cannot access local variable 'worker_state' where it is not associated with a value

Code snippet

from sanic import Sanic, text, json

app = Sanic("fuwu")

if __name__ == '__main__':
    app.run(host="0,0,0,0", port=8000, )

Expected Behavior

启动就报错，

How do you run Sanic?

as a model

Operating System

Windows

Sanic Version

sanic = {extras = ["ext"], version = "^24.6.0"}

Additional context

No response
@f754699 f754699 added the bug label Jul 13, 2024
@ChihweiLHBird
Copy link
Member

ChihweiLHBird commented Jul 18, 2024 •
edited
Loading 

from sanic import Sanic, text, json

app = Sanic("fuwu")

if __name__ == '__main__':
    app.run(host="0.0.0.0", port=8000)

Setting the correct host name seems to address the issue. But we might want to implement a host name validator to avoid confusing error message.

把host里的,改为.即可解决问题。同时建议在Linux或Unix系统上尝试

I am looking into it, the previous message above was incorrect.

pygeek added a commit to pygeek/sanic that referenced this issue Jul 31, 2024
@pygeek
sanic-orggh-2985: Fix startup serve reference error.
fe55023
@pygeek pygeek linked a pull request Jul 31, 2024 that will close this issue
Open
pygeek added a commit to pygeek/sanic that referenced this issue Jul 31, 2024
@pygeek
sanic-orggh-2985: Fixed startup serve UnboundLocalError.
aad408a
@pygeek
Copy link

pygeek commented Jul 31, 2024 •
edited
Loading

This is still a bug, however. mixins.startup.serve should still exit gracefully—fixed in my PR.

pygeek added a commit to pygeek/sanic that referenced this issue Jul 31, 2024
@pygeek
sanic-orggh-2985: Fixed mixins.startup.serve UnboundLocalError.
9ddf8bd
pygeek added a commit to pygeek/sanic that referenced this issue Jul 31, 2024
@pygeek
sanic-orggh-2985: Fixed mixins.startup.serve UnboundLocalError.
6d80c7e
pygeek added a commit to pygeek/sanic that referenced this issue Jul 31, 2024
@pygeek
sanic-orggh-2985: Fixed mixins.startup.serve UnboundLocalError.
0d7b2ae
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixed mixins.startup.serve UnboundLocalError. pygeek/sanic
3 participants
@pygeek @ChihweiLHBird @f754699