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

Aperta Adatta ai principianti
  • Qualcosa non va?
#257 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@Rishankgupta08 ci sta già lavorando.

Dal 14/11/2026.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
82/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
java
Ambito
testing-qa

Direzione di ricerca

Inizia dai tre punti indicati: ApiAuth.java intorno alla riga 117, ApiClient.java intorno alla riga 142 e ApiHealthChecker.java intorno alla riga 70, tutti sotto src/main/java/com/seleniumboot. In ciascuno trova il blocco catch che ingoia InterruptedException e aggiungi Thread.currentThread().interrupt() mantenendo la gestione esistente. È completato quando questi tre punti ripristinano il flag, il comportamento è altrimenti invariato e mvn test passa.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.

Lingua principale
Java
Stelle
12
Fork
6
Merge medio
10h
PR unite (30g)
76

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di seleniumboot/selenium-boot

Tutte le issue di seleniumboot/selenium-boot

Issue simili

Altre issue su Java

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.