Logging into a locally started instance clears its container reference, causing stop to do nothing

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#620 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
79/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
docker, go
Domaine
cli

Piste de recherche

Commence par cmd/login.go, où l’issue indique que login remplace le contexte enregistré, et suis la manière dont ce contexte est écrit. Reproduis le problème avec les trois commandes de l’issue en utilisant un fichier de configuration distinct, puis vérifie que login préserve le champ d’instance et que microcks stop arrête le conteneur ensuite.

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

Description

Describe the bug

After starting a local Microcks instance with the CLI, logging into the same server clears the instance reference from its saved context.

A subsequent microcks stop reports that no instance is associated with the context and exits successfully, even though the container is still running.

I reproduced this using a separate config file and an authentication-disabled Microcks instance.

Expected behavior

Logging into the same server should preserve its existing local instance reference. Running microcks stop afterward should stop the container created by microcks start.

Actual behavior

Before login, the saved context contains:
instance: microcks-login-repro
After login, it becomes:
instance: ""
Running stop then prints: No instance is associated with this context
The command exits with code 0, but Docker still reports the container as running. The Microcks API also continues to return HTTP 200.

How to Reproduce?

With Docker running, execute the following commands using a separate config file:

microcks start --name microcks-login-repro --port 18585 --rm --ready-timeout 120s --config repro-config.yaml
microcks login http://localhost:18585 --config repro-config.yaml
microcks stop --config repro-config.yaml

Check whether the container is still running:

docker inspect --format '{{.State.Running}}' microcks-login-repro

The output is: true
You can also compare repro-config.yaml before and after login to see that the context’s instance field has been cleared.
As a control, restoring the config saved before login and repeating the same stop command successfully stopped and removed the container.

To clean up manually after reproducing:

docker stop microcks-login-repro
Microcks version or git rev

36f1028f877a5f4502633f574aac627e7d25dc53

Install method (docker-compose, helm chart, operator, docker-desktop extension,...)

No response

Additional information

The likely cause is in cmd/login.go: login replaces the existing context with a ContextRef containing only Name, Server, and User, leaving out the Instance field previously saved by start.

Langage dominant
Go
Étoiles
59
Forks
75
Merge moyen
4 j 5 h
PR mergées (30 j)
15

Préparer son environnement

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.

Autres issues de microcks/microcks-cli

Toutes les issues de microcks/microcks-cli

Issues similaires

Plus d'issues Go

Recevez les nouvelles issues par e-mail

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