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

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#257 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@Rishankgupta08 đang làm issue này rồi.

Từ ngày 14/11/2026.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
82/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
java
Lĩnh vực
testing-qa

Hướng nghiên cứu

Bắt đầu với ba vị trí được nêu: ApiAuth.java quanh dòng 117, ApiClient.java quanh dòng 142 và ApiHealthChecker.java quanh dòng 70, tất cả nằm dưới src/main/java/com/seleniumboot. Ở mỗi nơi, tìm khối catch đang nuốt InterruptedException và thêm Thread.currentThread().interrupt(), đồng thời giữ nguyên cách xử lý hiện có. Hoàn thành khi ba vị trí này khôi phục cờ ngắt, hành vi còn lại không thay đổi và mvn test chạy thành công.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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.

Ngôn ngữ chính
Java
Star
12
Fork
6
Merge trung bình
10 giờ
Pull request đã merge (30 ngày)
76

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của seleniumboot/selenium-boot

Tất cả issue của seleniumboot/selenium-boot

Issue tương tự

Thêm issue về Java

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.