use getenv in configs
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 25/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Cần làm rõ
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- docker, shell
- Lĩnh vực
- devops, infrastructure
Hướng nghiên cứu
Bắt đầu bằng cách xác định cấu hình image Docker và phần khởi tạo biến môi trường hiện có ghi các giá trị vào config.php. Xác định những cài đặt cần cho hoạt động liên tục vẫn còn được hardcode và đánh giá liệu getenv có thể cung cấp chúng một cách an toàn hay không, bao gồm các trường hợp secret của container và xoay vòng. Được xem là hoàn tất khi các cài đặt liên quan luôn đọc biến môi trường mà không loại bỏ khả năng hỗ trợ cấu hình thủ công trong config.php.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
How to use GitHub
- Please use the 👍 reaction to show that you are interested into the same feature.
- Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
- Subscribe to receive notifications on status change and new comments.
Is your feature request related to a problem? Please describe.
Simply put, I can understand using the environment variables for initialising NC, but then not using environment variables for continued operations seems silly to me. Why hardcode the values when they are initially read from environment variables anyway?
Additionally using environment variables instead of hardcoding would allow using container secrets directly.
Additionally rotating DB passwords or locations becomes a breeze since it would all be done through environment variables.
Describe the solution you'd like
Star using: 'var' => getenv('ENV_VAR') as much as possible since manually editing config.php can be done eitherway.
Describe alternatives you've considered
Doing it myself, while yes I've done this. I see that this would be likely to benefit other people as well.
Additional context
Since this is an "advanced" image when compared to AIO I can understand potential hesitancy to adopt my suggestions. Yet I believe that it will help since it doesn't detract from original deployment methods in any way.
- Ngôn ngữ chính
- Shell
- Star
- 7.4k
- Fork
- 1.9k
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của nextcloud/docker
-
1. To develop bug feature: cron image
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Drop paragarph about anonymous (unnamed) volumes under Persistent Storage section fo the docsĐang mởbug docs
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
-
0. Needs triage bug
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
-
Imagick architecture mismatch (x86_64 vs aarch64) on official arm64 image causes Apache segfaultsĐang mở0. Needs triage bug
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 50/100
Tất cả issue của nextcloud/docker
Issue tương tự
-
PacReq: mangowcĐang mởpackage request
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
pacstall/pacstall-programs#9475 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 71/100
t4t5/omdrop-owl#14 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 1 ngày