Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Android: ProxyDetectorImpl crashes when DefaultProxySelector contains an invalid proxy port

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#13,052 4 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 2 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
78/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
android, java
Lĩnh vực
backend-api-design, networking

Hướng nghiên cứu

Bắt đầu trong core/src/main/java/io/grpc/internal/ProxyDetectorImpl.java, quanh detectProxy và các dòng được xác định qua việc kiểm tra mã nguồn, sau đó theo dõi cách DnsNameResolver gọi nó. Tái hiện trường hợp với https.proxyPort nằm ngoài phạm vi và kích hoạt phân giải tên. Hoàn tất khi IllegalArgumentException từ ProxySelector.select() không còn làm resolver bị crash và trạng thái proxy không hợp lệ chuyển về kết nối trực tiếp.

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

Mô tả

What version of gRPC-Java are you using?

1.66.0

Source inspection indicates that latest 1.84.0 is also affected.

What is your environment?

Android production application.

Most events are from Android 13 devices (89%), predominantly Xiaomi (85%).

What did you expect to see?

A malformed system proxy configuration should not cause an uncaught exception on the gRPC resolver executor.

Because the configured proxy cannot be used, gRPC should preferably treat it as no usable proxy and continue with a direct connection. At minimum, it should convert the exception into an ordinary name-resolution failure instead of terminating the process.

What did you see instead?

Android’s DefaultProxySelector throws IllegalArgumentException while constructing a proxy address with an out-of-range port. The exception escapes through ProxyDetectorImpl and becomes a fatal process crash.

Fatal Exception: java.lang.IllegalArgumentException: port out of range:899858473
    at java.net.InetSocketAddress.checkPort(InetSocketAddress.java:154)
    at java.net.InetSocketAddress.createUnresolved(InetSocketAddress.java:279)
    at sun.net.spi.DefaultProxySelector$1.run(DefaultProxySelector.java:315)
    at sun.net.spi.DefaultProxySelector$1.run(DefaultProxySelector.java:219)
    at java.security.AccessController.doPrivileged(AccessController.java:46)
    at sun.net.spi.DefaultProxySelector.select(DefaultProxySelector.java:218)
    at io.grpc.internal.ProxyDetectorImpl.detectProxy(ProxyDetectorImpl.java:230)
    at io.grpc.internal.ProxyDetectorImpl.proxyFor(ProxyDetectorImpl.java:200)
    at io.grpc.internal.DnsNameResolver.detectProxy(DnsNameResolver.java:269)
    at io.grpc.internal.DnsNameResolver.access$600(DnsNameResolver.java:66)
    at io.grpc.internal.DnsNameResolver$Resolve.run(DnsNameResolver.java:310)
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1100)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
    at java.lang.Thread.run(Thread.java:1572)

We observed multiple affected devices with different invalid ports. Each invalid value remains stable for its affected device. This suggests malformed device/system proxy state rather than corruption inside gRPC.

Analysis

On Android 13, DefaultProxySelector.select() reads the process proxy properties and passes the configured port directly to InetSocketAddress.createUnresolved().

InetSocketAddress rejects ports outside 0..65535.

The invalid value is the system proxy port, not the gRPC destination port. Although gRPC does not produce the malformed configuration, ProxyDetectorImpl currently allows the platform exception to escape from the resolver task.

The existing handling for a ProxySelector returning null or an empty list does not cover this case because the exception is thrown inside ProxySelector.select().

Comparable OkHttp behavior

OkHttp’s current implementation handles this exact case by catching IllegalArgumentException from ProxySelector.select() and treating it as no usable proxy, falling back to Proxy.NO_PROXY.

Its source explicitly explains the reason:

// A misconfigured system proxy (such as one with no port set) can make
// ProxySelector.select() itself throw IllegalArgumentException; treat that
// as "no usable proxy" rather than letting it crash.

Applying equivalent handling in gRPC would provide consistent behavior between regular OkHttp traffic and the gRPC OkHttp transport.

Reproduction

We have not reproduced the original malformed Android proxy state locally.

A synthetic reproduction should be possible by configuring:

https.proxyHost=<non-empty host>
https.proxyPort=<integer greater than 65535>

and then triggering name resolution through a gRPC channel using the default proxy detector.

Suggested change

Catch IllegalArgumentException around ProxySelector.select() and treat the result as no proxy:

final List<Proxy> proxies;
try {
  proxies = proxySelector.select(uri);
} catch (IllegalArgumentException e) {
  log.log(
      Level.WARNING,
      "ProxySelector failed while selecting a proxy; proceeding without proxy",
      e);
  return null;
}

Returning null from ProxyDetectorImpl.detectProxy() preserves connectivity through a direct connection and matches OkHttp’s behavior for malformed system proxy configuration.

If silently falling back to direct connectivity is considered inappropriate, converting the exception to IOException would still prevent a fatal process crash, although the channel would remain unavailable while the malformed proxy configuration persists.

Ngôn ngữ chính
Java
Star
12.1k
Fork
4k
Merge trung bình
2 ngày 3 giờ
Pull request đã merge (30 ngày)
30

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 grpc/grpc-java

Tất cả issue của grpc/grpc-java

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.