Cleanup: restore the interrupt flag in the API client and health checker

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

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

@Rishankgupta08 y travaille déjà.

Depuis le 14/11/2026.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
82/100
Type d'issue
Refactorisation
Clarté
Clairement spécifiée
Activité
Active
Stack technique
java
Domaine
testing-qa

Piste de recherche

Commencez par les trois emplacements indiqués : ApiAuth.java vers la ligne 117, ApiClient.java vers la ligne 142 et ApiHealthChecker.java vers la ligne 70, tous sous src/main/java/com/seleniumboot. Dans chacun, repérez le bloc catch qui avale InterruptedException et ajoutez Thread.currentThread().interrupt() en conservant le traitement existant. C'est terminé lorsque ces trois emplacements restaurent le drapeau, que le comportement est par ailleurs inchangé et que mvn test passe.

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

Description

good first issue

Part of the cleanup cycle from #251. SonarCloud rule java:S2142 flags these spots.

These places catch InterruptedException (often inside catch (Exception e)) and carry on without restoring the thread's interrupt flag. Anything further up that checks Thread.interrupted() (a parallel runner shutting down, a timeout) never sees the interrupt.

Where (line numbers as of master 2a07bba):

  • src/main/java/com/seleniumboot/client/ApiAuth.java: line 117
  • src/main/java/com/seleniumboot/client/ApiClient.java: line 142
  • src/main/java/com/seleniumboot/precondition/ApiHealthChecker.java: line 70

Fix: restore the flag and keep the existing behaviour. For a combined catch, split it out:

} catch (InterruptedException e) {
    Thread.currentThread().interrupt();
    // then the same handling the old catch did (log / wrap / return)
} catch (Exception e) {
    ...
}

Done when: the listed spots restore the interrupt flag, the behaviour is otherwise unchanged, and mvn test passes. Keep this PR to this package. SonarCloud reruns on the PR.

Langage dominant
Java
Étoiles
12
Forks
6
Merge moyen
10 h
PR mergées (30 j)
76

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 seleniumboot/selenium-boot

Toutes les issues de seleniumboot/selenium-boot

Issues similaires

Plus d'issues Java

Recevez les nouvelles issues par e-mail

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