Minimum port 5038 for 'user managed adb' prevents use of the default port 5037 which may be desired [301311826]

Fixed

Bug

Status Update

No update yet.

Description

m....@chkpowerquality.com.au

created issue #1

Sep 21, 2023 07:01AM

The minimum is in AdbOptionsService.java:

  /**
   * Default user managed adb port. Same as {@link #USER_MANAGED_ADB_PORT_MIN_VALUE}.
   */
  public static final int USER_MANAGED_ADB_PORT_DEFAULT = 5038;

  /**
   * Default adb port +1 to avoid conflict.
   */
  static final int USER_MANAGED_ADB_PORT_MIN_VALUE = 5038;

  /**
   * Max ephemeral port number for most modern operating systems.
   */
  static final int USER_MANAGED_ADB_PORT_MAX_VALUE = 65535;

If the user has an adb server running on 5037 already, and they want Android Studio to use it, that is not possible because of this minimum. It appears to be an oversight as the comment describes the value as a default.

The minimum valid TCP port is 1.

Comments

bi...@google.com <bi...@google.com> Sep 21, 2023 06:44PM

Assigned to an...@google.com.

ac...@google.com <ac...@google.com> Sep 21, 2023 09:12PM

Reassigned to rp...@google.com.

sa...@google.com <sa...@google.com> #2Sep 25, 2023 07:38PM

Fix on the way to allow default ADB port (5037) to be user managed.

Message last modified on Oct 6, 2023 12:02AM

sa...@google.com <sa...@google.com> #3Oct 4, 2023 07:23PM

Marked as fixed.

For internal records, we have patched this issue to allow adb server running on port 5037 to be user managed (see ag/24869854).

an...@google.com <an...@google.com> #4Oct 5, 2023 09:26PM

Thank you for your patience while our engineering team worked to resolve this issue. A fix for this issue is now available in:

Android Studio Iguana | 2023.2.1 Canary 7
Android Gradle Plugin 8.3.0-alpha07

We encourage you to try the latest update.

If you notice further issues or have questions, please file a new bug report.

Thank you for taking the time to submit feedback — we really appreciate it!

Issue 301311826