Cut the pre-auth-required attribute for now.

Author: jdcormie

binder/src/main/java/io/grpc/binder/ApiConstants.java

@@ -46,17 +46,6 @@ private ApiConstants() {}
   public static final NameResolver.Args.Key<UserHandle> TARGET_ANDROID_USER =
       NameResolver.Args.Key.create("target-android-user");
 
-  /**
-   * Marks an {@link io.grpc.EquivalentAddressGroup} as needing pre-authorization.
-   *
-   * <p>Clients should authorize servers before connecting to them, but older versions of the binder
-   * transport didn't do so. While this important extra security check is now possible (see {@link
-   * BinderChannelBuilder#preAuthorizeServers(boolean)}, it remains optional, because it's a slight
-   * behavior change and has a small performance cost and we don't want to break existing apps.
-   */
-  public static final Attributes.Key<Void> PRE_AUTH_REQUIRED =
-      Attributes.Key.create("pre-auth-required");
-
   /**
    * The authentic ServiceInfo for an {@link io.grpc.EquivalentAddressGroup} of {@link
    * AndroidComponentAddress}es, in case a {@link NameResolver} has already looked it up.

binder/src/main/java/io/grpc/binder/internal/BinderTransport.java

@@ -611,9 +611,7 @@ public BinderClientTransport(
       this.securityPolicy = factory.securityPolicy;
       this.offloadExecutor = offloadExecutorPool.getObject();
       this.readyTimeoutMillis = factory.readyTimeoutMillis;
-      this.preAuthorizeServer =
-          factory.preAuthorizeServers
-              || (options.getEagAttributes().get(ApiConstants.PRE_AUTH_REQUIRED) != null);
+      this.preAuthorizeServer = factory.preAuthorizeServers;
       numInUseStreams = new AtomicInteger();
       pingTracker = new PingTracker(Ticker.systemTicker(), (id) -> sendPing(id));