amihaiemil / amihaiemil/docker-java-api

Remove SslHttpClient?

Đang mở
#215 11 bình luận 0 reaction 0 người được giao Xem trên GitHub
bug postponed question
Ngôn ngữ chính
Java
Star
274
Fork
56
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

I would like to remove this class and all the code (constructors in ``RemoteDocker``, mostly) that makes any assumption regarding securing communication with a remote Docker instance.

My reasoning is that we shouldn't make any assumptions since we will most likely not cover all the possibilities and we are just polluting the library. If the user really needs it and knows what they are doing, they can provide a properly configured ``HttpClient`` when instantiating a ``RemoteDocker``.

What do you think?

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

Bắt đầu bằng cách xác định SslHttpClient và các constructor trong RemoteDocker, sau đó truy vết cách chúng giả định việc giao tiếp an toàn với một phiên bản Docker từ xa. Xem lại phần thảo luận trong các comment trước khi quyết định phạm vi. Hoàn thành có nghĩa là các giả định đó và mã liên quan được loại bỏ, đồng thời các caller có thể cung cấp một HttpClient đã được cấu hình.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
docker, java
Lĩnh vực
api
Loại issue
Tái cấu trúc
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Cần làm rõ
Mức phù hợp với người mới
30/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.