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

修复一些小问题 并 修复了Dockerfile #44

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sxhoio
Copy link

@sxhoio sxhoio commented Aug 19, 2024

首先是Dockerfile
按照仓库内出现的Dockerfile 他是无法使用的 所以我做了一点点修改 来让他可以运行

部分时候程序会崩溃所以做了写修改

@sml2h3
Copy link
Owner

sml2h3 commented Aug 20, 2024

关于file.size那块可能得再考虑一下,我记得测试的时候直接判断为None会有问题

@sxhoio sxhoio closed this Sep 24, 2024
@sxhoio sxhoio reopened this Sep 24, 2024
@sxhoio
Copy link
Author

sxhoio commented Sep 24, 2024

感谢回复 主要是我在测试的时候发现
image
会提示 'UploadFile' object has no attribute 'size'
所以我才会值判断是否为空 而且是大多数图像都会这样显示
image
在修改了None 可以正常使用
引起的主要原因貌似是file.size读取不到的问题
但是暂时不确定是因为版本还是支持库版本问题

@sxhoio
Copy link
Author

sxhoio commented Sep 24, 2024

同时Docker 部署的情况下大部分时间用不到热重载所以我没有直接调用main.py文件在Dockerfile里

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

Successfully merging this pull request may close these issues.

2 participants