Check `NetworkRequest#mDestroyed` before each network request?
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đá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
- 72/100
Hướng nghiên cứu
Bắt đầu tại AidlNetworkRequest#performNetworkRequestV2 và so sánh với kiểm tra NetworkRequest#mDestroyed hiện có được thực hiện trong connect. Xác minh rằng việc gọi close() trước một request được phát hiện ở đầu operation và dẫn đến một IllegalStateException có ý nghĩa thay vì lỗi null-context xảy ra về sau.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
I recently ran into an issue while trying to make a network request on an NextcloudAPI instance, on which I called close() before.
Of course, it failed, but at an odd place, that I didn't expected (I think context was null at some place).
This made me wondering why we do check for NetworkRequest#mDestroyed when connecting, but not before each request. Wouldn't make it sense to add this check also at the beginning of AidlNetworkRequest#performNetworkRequestV2 and throw a more meaningful IllegalStateException or something alike?
cc @David-Development
- Ngôn ngữ chính
- Java
- Star
- 75
- Fork
- 36
- Merge trung bình
- 23 giờ 43 phút
- Pull request đã merge (30 ngày)
- 14
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của nextcloud/Android-SingleSignOn
-
0. Needs triage bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
nextcloud/Android-SingleSignOn#1081 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
0. Needs triage bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 45/100
nextcloud/Android-SingleSignOn#894 · 1 reaction ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[Bug]: Brotli or ZSTD reverse proxy compression breaks SSO authentication in some android appsĐang mở0. Needs triage bug
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
nextcloud/Android-SingleSignOn#859 · 3 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
0. Needs triage enhancement
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
nextcloud/Android-SingleSignOn#849 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
0. Needs triage enhancement
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 30/100
nextcloud/Android-SingleSignOn#771 · 3 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của nextcloud/Android-SingleSignOn
Issue tương tự
-
type: possible bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
grimmory-tools/grimmory#2850 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 90/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
aoqia194/leaf-loader#19 ·