amihaiemil / amihaiemil/docker-java-api

Remove SslHttpClient?

オープン
#215 コメント 11 件 リアクション 0 件 担当者 0 名 GitHub で見る
bug postponed question
主要言語
Java
スター
274
フォーク
56
PR マージ指標
30日以内にマージされた PR はありません

説明

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?

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

まず SslHttpClient と RemoteDocker のコンストラクターを見つけ、次にそれらがリモートの Docker インスタンスとの安全な通信をどのように前提としているかを追跡します。範囲を決める前に、コメントの議論を確認してください。これらの前提と関連コードが削除され、呼び出し元が構成済みの HttpClient を提供できるようになれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
docker, java
領域
api
issue の種類
リファクタリング
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
30/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。