-
Notifications
You must be signed in to change notification settings - Fork 141
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
Tạo file setting để thay thế cho biến môi trường #22
Comments
Mình cũng băn khoăn là good practice thì nên xử lý setting như nào. Hiện mình dùng biến môi trường vì nghĩ rằng khi chuyển sang Docker sẽ dễ dàng hơn ? |
chỉ dễ dàng hơn với một vài giá trị thôi, chứ như đã phân tích, biến môi trường bị hạn chế khoản phản ánh kiểu dữ liệu, nên nếu vấp phải vấn đề này thì chữ "dễ" sẽ biến thành "khó", chưa kể nếu số lượng setting lớn thì quản lý sẽ mệt (không thể gom tổ chức thành nhóm). |
Ukie, mình đã chuyển thành task trong backlog |
Thấy bạn đang dùng các biến môi trường làm settings.
Điều này hơi kì khôi. Biến môi trường chỉ nên dùng một ít để overwrite settings thôi. Biến môi trường có hạn chế:
"1"
,"true"
) dẫn tới bạn phải đặt ra luật riêng để "ép kiểu" từ text sang giá trị thực, ví dụ từ "1" thành integer, từ "true", "1", "yes" thành boolean.USERNAME='admin'
, không thể viết cách cho thông thoáng, dễ nhìnUSERNAME = admin
.Đề xuất giải pháp:
Ghi chú:
người ta đặt ra một đống quy ước để biểu diễn giá trị boolean ("0", "1", "true", "false", "yes", "no", "t", "f"...)
The text was updated successfully, but these errors were encountered: