testcontainers / testcontainers/testcontainers-python

Investigate what to do about "localnpipe" on windows

Ouverte
#415 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
Python
Étoiles
2.3k
Forks
386
Merge moyen
4 h 40 min
PR mergées (30 j)
1

Description

it has been some time. the urllib3/docker-py incompatibilities are fixed. the time of using named pipes on windows is here. the question is how to deal with them in this library. for now, i am suggesting a fix like:

class DockerContainer:
    # ...
    def get_container_host_ip(self) -> str:
        # ...
        # see https://github.com/testcontainers/testcontainers-python/issues/415
        if host == "localnpipe" and "Windows" == system():
            return "localhost"
        # ...

but it really should get investigated in the short term.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par DockerContainer.get_container_host_ip et examinez la manière dont la bibliothèque gère actuellement les valeurs d’hôte sous Windows, en tenant compte du contexte de compatibilité urllib3/docker-py. Reproduisez le cas localnpipe sous Windows ; l’investigation est terminée lorsque le comportement pris en charge et la modification requise sont clairement documentés et validés.

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

Évaluation

Stack technique
docker, python
Domaine
devtools, operating-systems
Type d'issue
Fonctionnalité
Difficulté
5/5
Temps estimé
Plus d'une semaine
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
25/100

Recevez les nouvelles issues par e-mail

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