-
-
Notifications
You must be signed in to change notification settings - Fork 324
File not found error on page refresh (0.28.0) #389
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
Comments
@brentbaum @ksmoore17 I'm having trouble reproducing this issue, can you reply with a basic script or set of instructions for doing so? Once I have that I can write a test to prevent this regression in the future. |
This issue isn't happening on 0.29.0 for me. |
Awesome! Feel free to re-open if this crops up again. |
I am getting this issue again, running on latest version 0.34.0 of idom: python .\filterable.py
2022-01-16T10:25:37+0100 | INFO | Using SanicServer
[2022-01-16 10:25:37 +0100] [8848] [INFO] Goin' Fast @ http://127.0.0.1:8000
[2022-01-16 10:25:37 +0100] [8848] [WARNING] Sanic tried to use loop.add_signal_handler but it is not implemented on this platform.
[2022-01-16 10:25:37 +0100] [8848] [WARNING] Sanic tried to use loop.add_signal_handler but it is not implemented on this platform.
[2022-01-16 10:25:37 +0100] [8848] [INFO] Starting worker [8848]
[2022-01-16 10:25:45 +0100] - (sanic.access)[INFO][127.0.0.1:60219]: GET http://127.0.0.1:8000/ 302 0
[2022-01-16 10:25:45 +0100] [8848] [ERROR] Exception occurred while handling uri: 'http://127.0.0.1:8000/client/index.html'
Traceback (most recent call last):
File "C:\Users\acivi\AppData\Local\Programs\Python\Python39\lib\site-packages\sanic\static.py", line 87, in _handler
stats = await stat(file_path)
File "C:\Users\acivi\AppData\Local\Programs\Python\Python39\lib\site-packages\aiofiles\os.py", line 13, in run
return await loop.run_in_executor(executor, pfunc)
File "C:\Users\acivi\AppData\Local\Programs\Python\Python39\lib\concurrent\futures\thread.py", line 52, in run
result = self.fn(*self.args, **self.kwargs)
FileNotFoundError: [WinError 2] The system cannot find the file specified: 'C:\\Users\\acivi\\AppData\\Local\\Programs\\Python\\Python39\\lib\\site-packages\\idom-0.34.0-py3.9.egg\\idom\\client\\index.html'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\acivi\AppData\Local\Programs\Python\Python39\lib\site-packages\sanic\app.py", line 942, in handle_request
response = await response
File "C:\Users\acivi\AppData\Local\Programs\Python\Python39\lib\site-packages\sanic\static.py", line 132, in _handler
raise FileNotFound(
sanic.exceptions.FileNotFound: File not found
[2022-01-16 10:25:45 +0100] - (sanic.access)[INFO][127.0.0.1:60219]: GET http://127.0.0.1:8000/client/index.html 404 21 |
Is this specific to the Sanic server implementation or does this occur on other implementations? |
I'm not able to reproduce at the moment. It's possible this is a Windows specific issue. Happy to hop on a call to try and sort this out if the issue persists. |
@acivitillo is this still happening at all? |
There might be some problems with Windows, but this specific one did not
show up anymore.
…On Fri, Jan 28, 2022 at 3:35 AM Ryan Morshead ***@***.***> wrote:
@acivitillo <https://github.com/acivitillo> is this still happening at
all?
—
Reply to this email directly, view it on GitHub
<#389 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ANDDBTC3NKDW42HO3JDABALUYH6FJANCNFSM46M73OPA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I'm going to close this for now. If it comes up again feel free to comment. |
With idom 0.28.0[stable] a regression was introduced where refreshing the page yields a
Error: File not found
error. Downgrading to 0.27.0 fixes the issue.This appears to be resolved in
main
now, but we should introduce a test to catch this in the future.Originally posted by @brentbaum in #388
The text was updated successfully, but these errors were encountered: