amihaiemil / amihaiemil/docker-java-api

Remove SslHttpClient?

Ouverte
#215 11 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
bug postponed question
Langage dominant
Java
Étoiles
274
Forks
56
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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?

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par localiser SslHttpClient et les constructeurs de RemoteDocker, puis retracez la manière dont ils supposent une communication sécurisée avec une instance Docker distante. Consultez la discussion dans les commentaires avant de définir le périmètre. Le travail est terminé lorsque ces suppositions et le code associé sont supprimés, tout en permettant aux appelants de fournir un HttpClient configuré.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
docker, java
Domaine
api
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
30/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.