uninstall.sh: prints 'Deleted bucket' after a failed bucket delete

Ouverte Adaptée aux débutants
#133 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é
1/5
Temps estimé
Moins d'une heure
Accessibilité débutants
94/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
aws, shell
Domaine
cli, cloud

Piste de recherche

Ouvrez uninstall.sh aux lignes 789–790 et examinez la commande de suppression du bucket ainsi que ses messages de réussite et d’avertissement. Vérifiez les tests de désinstallation à proximité ou les usages éventuels ; le payload ne nomme aucun fichier de test. Le travail est terminé lorsqu’un aws s3 rb en échec affiche uniquement l’avertissement et qu’une suppression réussie affiche la confirmation.

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

Description

bug good first issue

uninstall.sh L789-790:

aws s3 rb "s3://${b}" --force --region "$SCAN_REGION" 2>/dev/null || warn "Could not delete ${b}"
ok "Deleted bucket: ${b}"

Problem: ok always runs. If the delete fails (for example AccessDenied), the user sees "Could not delete X" and then "Deleted bucket: X" for the same bucket.

Fix:

if aws s3 rb "s3://${b}" --force --region "$SCAN_REGION" 2>/dev/null; then
  ok "Deleted bucket: ${b}"
else
  warn "Could not delete ${b}"
fi

Found by a code read at commit 1e50295. The code was read, but not run on a real instance.

Langage dominant
Shell
Étoiles
73
Forks
11
Merge moyen
1 min
PR mergées (30 j)
2

Préparer son environnement

Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.

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 inceptionstack/lowkey

Toutes les issues de inceptionstack/lowkey

Issues similaires

Plus d'issues Shell/Bash

Recevez les nouvelles issues par e-mail

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