feat: Support transport and binding-enforcement MDS parameters. #1558

rmehta19 · 2024-10-26T02:06:20Z

No description provided.

rmehta19 · 2024-10-26T02:08:41Z

westarle · 2024-10-28T12:18:59Z

oauth2_http/java/com/google/auth/oauth2/ComputeEngineCredentials.java

@@ -696,6 +725,14 @@ public Collection<String> getDefaultScopes() {
      return defaultScopes;
    }

+    public String getTransport() {


Please provide javadoc on these, pointing to publicly documented Metadata Service parameters.

Discussed offline: We will update public MDS docs when the feature is non-experimental. Since the usage of the S2A + MTLS + bound tokens flow is guarded by an environment variable (EXPERIMENTAL_GOOGLE_API_USE_S2A). Until the public docs are updated, we can link to AIP here and #1400 (AIP already linked in this merged PR).

rmehta19 · 2024-11-07T20:01:48Z

@lqiu96 , @zhumin8 : Would be able to take a look at this change and leave any feedback? Thanks!

sonarqubecloud · 2024-11-19T19:25:38Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

lqiu96 · 2024-12-03T17:10:05Z

oauth2_http/java/com/google/auth/oauth2/ComputeEngineCredentials.java

+    if (!transport.isEmpty()) {
+      tokenUrl.set("transport", transport);
+    }
+    if (!bindingEnforcement.isEmpty()) {
+      tokenUrl.set("binding-enforcement", bindingEnforcement);
+    }


I think this should be null and empty check. Setter can set a null value unless we want to add a check there

Thanks for pointing that out, done in 9d46bbd

lqiu96 · 2024-12-03T17:10:21Z

oauth2_http/java/com/google/auth/oauth2/ComputeEngineCredentials.java

+    private String transport = "";
+    private String bindingEnforcement = "";


nit:

Suggested change

private String transport = "";

private String bindingEnforcement = "";

private String transport;

private String bindingEnforcement;

Done in 3cb6e98

lqiu96 · 2024-12-03T17:11:04Z

Update to use conventional commits:
feat: Support transport and binding-enforcement MDS parameters

rmehta19 · 2024-12-03T23:38:18Z

Thanks for the review @lqiu96 !

oauth2_http/java/com/google/auth/oauth2/ComputeEngineCredentials.java

blakeli0 · 2024-12-13T18:51:37Z

oauth2_http/java/com/google/auth/oauth2/ComputeEngineCredentials.java

@@ -200,6 +218,16 @@ String createTokenUrlWithScopes() {
    if (!scopes.isEmpty()) {
      tokenUrl.set("scopes", Joiner.on(',').join(scopes));
    }
+    if (transport == Transport.MTLS) {
+      tokenUrl.set("transport", "mtls");


Fields can be added to the Enum class, so that each Enum can have its own label, then you can just do tokenUrl.set("transport", transport.getLabel()). See this example.

Thanks for pointing this out! Done in latest commit.

blakeli0 · 2024-12-13T18:57:06Z

oauth2_http/java/com/google/auth/oauth2/ComputeEngineCredentials.java

@@ -109,6 +109,16 @@ public class ComputeEngineCredentials extends GoogleCredentials
  static final int MAX_COMPUTE_PING_TRIES = 3;
  static final int COMPUTE_PING_CONNECTION_TIMEOUT_MS = 500;

+  public enum Transport {


I know the it is called transport in the token url, but since Transport in GAPIC has different meaning (the value of it is either gRPC or httpJson), can we name this differently? Maybe AuthTransport? Javadocs for what ALTS and MTLS mean would also be appreciated.

Renamed to AuthTransport in latest commit and added some javadocs. Thanks!

lqiu96 · 2024-12-19T21:50:07Z

oauth2_http/java/com/google/auth/oauth2/ComputeEngineCredentials.java

+    // Binding enforcement will always happen, irrespective of the IAM policy.
+    ON("on"),
+    // Binding enforcement will depend on IAM policy.
+    IAMPOLICY("iam-policy");


Suggested change

IAMPOLICY("iam-policy");

IAM_POLICY("iam-policy");

I believe the convention is underscore to separate the words for constants

Done in f36a19c

lqiu96 · 2024-12-19T21:51:35Z

oauth2_http/java/com/google/auth/oauth2/ComputeEngineCredentials.java

+    @CanIgnoreReturnValue
+    public Builder setTransport(AuthTransport transport) {
+      this.transport = transport;
+      return this;
+    }
+
+    @CanIgnoreReturnValue
+    public Builder setBindingEnforcement(BindingEnforcement bindingEnforcement) {
+      this.bindingEnforcement = bindingEnforcement;
+      return this;
+    }


can you some small/ simple javadocs for these public setters?

Done in 8a2d967

lqiu96 · 2024-12-19T21:52:12Z

oauth2_http/java/com/google/auth/oauth2/ComputeEngineCredentials.java

+    public AuthTransport getTransport() {
+      return transport;
+    }
+
+    public BindingEnforcement getBindingEnforcement() {
+      return bindingEnforcement;
+    }
+


same here. also if it's returning AuthTransport, can it be getAuthTransport()?

Done in 8a2d967

oauth2_http/javatests/com/google/auth/oauth2/ComputeEngineCredentialsTest.java

lqiu96 · 2024-12-19T21:56:33Z

oauth2_http/java/com/google/auth/oauth2/ComputeEngineCredentials.java

@@ -109,6 +109,40 @@ public class ComputeEngineCredentials extends GoogleCredentials
  static final int MAX_COMPUTE_PING_TRIES = 3;
  static final int COMPUTE_PING_CONNECTION_TIMEOUT_MS = 500;

+  public enum AuthTransport {


I think AuthTransport is ok for this. I would prefer if this could be more specific if possible.

Is there a term for alts and mtls type of transport that we can use? Does TlsTransport make sense? I don't know if TLS has a transport concept or if that would be confusing.

GoogleMtlsTransport? Though I'm not sure mtls/atls are google specific terms

TLDR: WDYT of GoogleAuthTransport?

I think AuthTransport is ok for this. I would prefer if this could be more specific if possible.

We generally call this "transport type": describes how the client is authenticating to Google APIS either via GFE (using mtls) or via DirectPath (using alts). However Blake mentioned above that transport has a different meaning in Client libraries (grpc vs http), so it would be good to clarify to something more specific, which is how we landed on AuthTransport initially.

Is there a term for alts and mtls type of transport that we can use? Does TlsTransport make sense? I don't know if TLS has a transport concept or if that would be confusing.

TLS is the security protocol being used, it's used in both mtls and alts. mtls is mutually authenticated TLS, alts is (in short) "Google-optimized" mtls (see public docs).

In the enum name, I think we want to focus on how clients are authenticating to Google APIS, not details of the protocol being used (mtls vs alts). How about GoogleAuthTransport? I think this captures that the enum is listing different ways to ways clients can authenticate to Google APIs.

GoogleMtlsTransport? Though I'm not sure mtls/atls are google specific terms.
I think this name would be ok. One thing is that mtls via GFE strictly adheres to the TLS standard (i.e. not Google specific). Whereas alts is a Google specific protocol.

I've done this in 8ce3a4d, let me know if the name sounds ok.

rmehta19 · 2025-01-07T19:12:44Z

Friendly ping @lqiu96, @blakeli0. Please let me know if there's anything else to address on this PR.

oauth2_http/java/com/google/auth/oauth2/ComputeEngineCredentials.java

lqiu96

Added a nit small request for a bit more javadocs on public enums, lgtm otherwise.

rmehta19 · 2025-01-09T14:19:33Z

Added a nit small request for a bit more javadocs on public enums, lgtm otherwise.

Thanks @lqiu96 ! Addressed. Would you be able to merge if the changes look good to you?

lqiu96 · 2025-01-09T15:28:08Z

oauth2_http/java/com/google/auth/oauth2/ComputeEngineCredentials.java

+  // Experimental Feature:
+  // Behavior of setting {@link GoogleAuthTransport} / {@link BindingEnforcement}:
+  // - MTLS-bound token where binding enforcement depends on IAM policy: MTLS / {}, {} / IAM_POLICY,
+  // MTLS / IAM_POLICY
+  // - MTLS-bound token where bindings are always enforced: {} / ON, MTLS / ON
+  // - DirectPath bound token: ALTS / {}


Can you move this into the javadocs for GoogleAuthTransport so it can be referenced for users in the future?

lqiu96 · 2025-01-09T15:30:16Z

Added a nit small request for a bit more javadocs on public enums, lgtm otherwise.

Thanks @lqiu96 ! Addressed. Would you be able to merge if the changes look good to you?

Yep lgtm, feel free to merge

rmehta19 · 2025-01-09T17:13:28Z

Yep lgtm, feel free to merge

Thanks @lqiu96 ! I don't have write permissions on this repo; would you be able to merge?

lqiu96 · 2025-01-09T19:52:36Z

oauth2_http/java/com/google/auth/oauth2/ComputeEngineCredentials.java

+  // Experimental Feature:
+  //


can you remove this?

Thanks for catching! removed.

oauth2_http/java/com/google/auth/oauth2/ComputeEngineCredentials.java

| Package | Type | Package file | Manager | Update | Change | |---|---|---|---|---|---| | [org.jetbrains:annotations](https://github.com/JetBrains/java-annotations) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `26.0.1` -> `26.0.2` | | [io.grpc:grpc-stub](https://github.com/grpc/grpc-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `1.69.1` -> `1.70.0` | | [io.grpc:grpc-protobuf](https://github.com/grpc/grpc-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `1.69.1` -> `1.70.0` | | [io.grpc:grpc-netty](https://github.com/grpc/grpc-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `1.69.1` -> `1.70.0` | | [io.grpc:protoc-gen-grpc-java](https://github.com/grpc/grpc-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `1.69.1` -> `1.70.0` | | [io.grpc:grpc-bom](https://github.com/grpc/grpc-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `1.69.1` -> `1.70.0` | | [io.grpc:grpc-api](https://github.com/grpc/grpc-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `1.69.1` -> `1.70.0` | | [com.google.api-client:google-api-client](https://github.com/googleapis/google-api-java-client) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.7.1` -> `2.7.2` | | [com.squareup.wire:wire-schema](https://github.com/square/wire) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `5.0.0` -> `5.2.1` | | [com.squareup.wire:wire-runtime](https://github.com/square/wire) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `5.0.0` -> `5.2.1` | | [com.squareup.wire:wire-reflector](https://github.com/square/wire) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `5.0.0` -> `5.2.1` | | [com.squareup.wire:wire-moshi-adapter](https://github.com/square/wire) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `5.0.0` -> `5.2.1` | | [com.squareup.wire:wire-grpc-client](https://github.com/square/wire) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `5.0.0` -> `5.2.1` | | [com.squareup.wire:wire-gradle-plugin](https://github.com/square/wire) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `5.0.0` -> `5.2.1` | | [com.squareup.wire:wire-bom](https://github.com/square/wire) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `5.0.0` -> `5.2.1` | | [com.google.auth:google-auth-library-oauth2-http](https://github.com/googleapis/google-auth-library-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `1.30.1` -> `1.31.0` | | [com.google.auth:google-auth-library-credentials](https://github.com/googleapis/google-auth-library-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `1.30.1` -> `1.31.0` | | [com.datadoghq:dd-trace-api](https://github.com/datadog/dd-trace-java) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `1.45.1` -> `1.45.2` | | [com.datadoghq:dd-trace-ot](https://github.com/datadog/dd-trace-java) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `1.45.1` -> `1.45.2` | | [software.amazon.awssdk:sdk-core](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.2` -> `2.30.4` | | [software.amazon.awssdk:dynamodb-enhanced](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.2` -> `2.30.4` | | [software.amazon.awssdk:dynamodb](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.2` -> `2.30.4` | | [software.amazon.awssdk:aws-core](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.2` -> `2.30.4` | | [software.amazon.awssdk:bom](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.2` -> `2.30.4` | | [software.amazon.awssdk:auth](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.2` -> `2.30.4` | --- ### Release Notes <details> <summary>JetBrains/java-annotations (org.jetbrains:annotations)</summary> ### [`v26.0.2`](https://github.com/JetBrains/java-annotations/blob/HEAD/CHANGELOG.md#Version-2602) [Compare Source](JetBrains/java-annotations@26.0.1...26.0.2) - Fixed missing klibs for apple artifacts. </details> <details> <summary>googleapis/google-api-java-client (com.google.api-client:google-api-client)</summary> ### [`v2.7.2`](https://github.com/googleapis/google-api-java-client/blob/HEAD/CHANGELOG.md#272-2025-01-22) ##### Bug Fixes - Add warnings to users about using credentials from external sources ([#2551](googleapis/google-api-java-client#2551)) ([3bb2879](googleapis/google-api-java-client@3bb2879)) </details> <details> <summary>square/wire (com.squareup.wire:wire-schema)</summary> ### [`v5.2.1`](https://github.com/square/wire/blob/HEAD/CHANGELOG.md#Version-521) [Compare Source](square/wire@5.2.0...5.2.1) *2025-01-07* ##### JVM generation - Fix support for mutable messages in Wire's Kotlin Generator. ([#3233](square/wire#3233) by \[Rahul Ravikumar]\[tikurahul]) ### [`v5.2.0`](https://github.com/square/wire/blob/HEAD/CHANGELOG.md#Version-520) [Compare Source](square/wire@5.1.0...5.2.0) *2025-01-06* ##### Common - Enforce recursion limit when parsing nested groups. ([#3119](square/wire#3119)) ##### CLI `wire-compiler` - It is now possible to set multiple targets. ([#3106](square/wire#3106) & [#3107](square/wire#3107)) - The option `opaque_types` introduced in `4.9.2` for the Wire Gradle plugin is now available on CLI. ([#3147](square/wire#3147)) ##### JVM generation - [KotlinPoet has been updated to `2.0.0`](https://square.github.io/kotlinpoet/changelog/#version-200) which dramatically changes how generated Kotlin files are wrapped. This is neither a source nor a binary breaking changes. - A new `@WireEnclosingType` annotation is now applied to generated types so R8 doesn't prune too much. ([#3123](square/wire#3123)) - Split the redact method into chunks when a type has more than 100 fields to avoid compilation error. ([#3214](square/wire#3214) by \[Damian Wieczorek]\[damianw]) - Add support for mutable messages in Wire's Kotlin Generator. ([#3217](square/wire#3217) by \[Rahul Ravikumar]\[tikurahul]) - You can opt-in by adding `mutableTypes = true` on your Kotlin target. This is unsafe and we do not recommend that you use it unless you have a sound use-case for it. - Wire is now using Palantir's JavaPoet instead of Square's JavaPoet. ##### Swift - Fix buffer overflow and data corruption when a type has more than 5 layers of nesting ([#3203](square/wire#3203) by \[Eric Amorde]\[amorde]) ### [`v5.1.0`](https://github.com/square/wire/blob/HEAD/CHANGELOG.md#Version-510) [Compare Source](square/wire@5.0.0...5.1.0) *2024-09-11* ##### Common - Support for Kotlin `2.0.20`. ([#3093](square/wire#3093)) - `srcDir(String)` has been undeprecated. ([#3039](square/wire#3039)) - Some loggings now happen at the debug level, instead of info. ([#3041](square/wire#3041)) - Remove some unactionable warnings on Kotlin/JS ([#3047](square/wire#3047)) - Propagate the deprecated flag on EnumType after pruning by wire-gradle-plugin ([#3076](square/wire#3076) by \[Aaron Edwards]\[aaron-edwards]) - Introduce `ProtoReader32`, a specialization for Kotlin/JS ([#3077](square/wire#3077)) This is an alternative to `ProtoReader`, which uses `Long` as a cursor. It originates as an optimization for Kotlin/JS, where `Long` cursors are prohibitively expensive. - Fix Gradle project isolation issue when reading a property ([#3078](square/wire#3078) by \[Aurimas]\[liutikas]) - Change the recursion limit to match grpc's default ([#3091](square/wire#3091)) ##### Kotlin - New enum option `enum_mode` to take precedence over the `enumMode` option added in `5.0.0-alpha02`. Use this if you want to migrate your enums granularly. ([#2993](square/wire#2993)) - Don't throw if reading trailers fail ([#3087](square/wire#3087)) ##### Swift - Avoid crash when parsing an empty repeated `[packed=true]` for fixed-length types. ([#3044](square/wire#3044) by \[Sasha Weiss]\[sashaweiss-signal]) </details> <details> <summary>googleapis/google-auth-library-java (com.google.auth:google-auth-library-oauth2-http)</summary> ### [`v1.31.0`](https://github.com/googleapis/google-auth-library-java/blob/HEAD/CHANGELOG.md#1310-2025-01-22) ##### Features - ImpersonatedCredentials to support universe domain for idtoken and signblob ([#1566](googleapis/google-auth-library-java#1566)) ([adc2ff3](googleapis/google-auth-library-java@adc2ff3)) - Support transport and binding-enforcement MDS parameters. ([#1558](googleapis/google-auth-library-java#1558)) ([9828a8e](googleapis/google-auth-library-java@9828a8e)) ##### Documentation - Promote use of bill of materials in quickstart documentation ([#1620](googleapis/google-auth-library-java#1620)) ([fc20d9c](googleapis/google-auth-library-java@fc20d9c)), closes [#1552](googleapis/google-auth-library-java#1552) </details> <details> <summary>datadog/dd-trace-java (com.datadoghq:dd-trace-api)</summary> ### [`v1.45.2`](https://github.com/DataDog/dd-trace-java/releases/tag/v1.45.2): 1.45.2 ##### Components ##### Application Security Management (WAF) - 🐛 🍒 8258 - Prevents a NPE when there is no subscriber for user events ([#8260](DataDog/dd-trace-java#8260) - [@manuel-alvarez-alvarez](https://github.com/manuel-alvarez-alvarez)) </details> --- ### Configuration 📅 **Schedule**: Branch creation - "after 6pm every weekday,before 2am every weekday" in timezone Australia/Melbourne, Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 👻 **Immortal**: This PR will be recreated if closed unmerged. Get [config help](https://github.com/renovatebot/renovate/discussions) if that's undesired. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate). GitOrigin-RevId: 12db0f59db2e6ebf55203c87fccab042d495106a

rmehta19 requested review from a team as code owners October 26, 2024 02:06

product-auto-label bot added the size: s Pull request size is small. label Oct 26, 2024

Support transport and binding-enforcement MDS parameters.

91dba11

rmehta19 force-pushed the mds-cred-token-req-param branch from 04db8a3 to 91dba11 Compare October 26, 2024 02:07

westarle reviewed Oct 28, 2024

View reviewed changes

add some tests.

ba11e8a

product-auto-label bot added size: m Pull request size is medium. and removed size: s Pull request size is small. labels Oct 28, 2024

Merge branch 'main' into mds-cred-token-req-param

47ec152

lqiu96 reviewed Dec 3, 2024

View reviewed changes

rmehta19 changed the title ~~Support transport and binding-enforcement MDS parameters.~~ feat: Support transport and binding-enforcement MDS parameters. Dec 3, 2024

rmehta19 added 2 commits December 3, 2024 15:33

null or empty.

9d46bbd

no default value in builder.

3cb6e98

Merge branch 'main' into mds-cred-token-req-param

cb70556

lqiu96 reviewed Dec 4, 2024

View reviewed changes

oauth2_http/java/com/google/auth/oauth2/ComputeEngineCredentials.java Show resolved Hide resolved

rmehta19 added 2 commits December 13, 2024 09:30

Merge branch 'main' into mds-cred-token-req-param

4d0440b

use enums to hold possible Transport + BindingEnforcement types

c96df22

blakeli0 reviewed Dec 13, 2024

View reviewed changes

rmehta19 added 2 commits December 13, 2024 14:54

Merge branch 'main' into mds-cred-token-req-param

a625889

Enum labels + Transport -> AuthTransport.

3d46963

lqiu96 reviewed Dec 19, 2024

View reviewed changes

oauth2_http/javatests/com/google/auth/oauth2/ComputeEngineCredentialsTest.java Show resolved Hide resolved

lqiu96 reviewed Dec 19, 2024

View reviewed changes

rmehta19 added 5 commits December 20, 2024 18:45

separate constant with underscore.

f36a19c

add javadocs + getTransport -> getAuthTransport.

8a2d967

add test both params null.

ce74e45

AuthTransport -> GoogleAuthTransport.

8ce3a4d

Merge branch 'main' into mds-cred-token-req-param

838380f

lqiu96 reviewed Jan 7, 2025

View reviewed changes

oauth2_http/java/com/google/auth/oauth2/ComputeEngineCredentials.java Show resolved Hide resolved

lqiu96 approved these changes Jan 7, 2025

View reviewed changes

rmehta19 added 2 commits January 9, 2025 05:55

Merge branch 'main' into mds-cred-token-req-param

e921991

add some javadoc.

20964e2

lqiu96 reviewed Jan 9, 2025

View reviewed changes

move javadoc to GoogleAuthTransport javadoc.

94618e0

lqiu96 reviewed Jan 9, 2025

View reviewed changes

oauth2_http/java/com/google/auth/oauth2/ComputeEngineCredentials.java Show resolved Hide resolved

clean up comments.

9261fe8

lqiu96 merged commit 9828a8e into googleapis:main Jan 9, 2025
15 checks passed

release-please bot mentioned this pull request Jan 9, 2025

chore(main): release 1.31.0 #1608

Merged

This was referenced Jan 16, 2025

chore(setup-release-please): release 1.31.0 #1617

Closed

chore(setup-release-please): release 1.31.0 #1618

Closed

rockspore mentioned this pull request Jan 17, 2025

feat: enable DirectPath bound token in InstantiatingGrpcChannelProvider googleapis/sdk-platform-java#3572

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: Support transport and binding-enforcement MDS parameters. #1558

feat: Support transport and binding-enforcement MDS parameters. #1558

rmehta19 commented Oct 26, 2024

rmehta19 commented Oct 26, 2024

westarle Oct 28, 2024

rmehta19 Oct 30, 2024

rmehta19 commented Nov 7, 2024

sonarqubecloud bot commented Nov 19, 2024

lqiu96 Dec 3, 2024

rmehta19 Dec 3, 2024

lqiu96 Dec 3, 2024

rmehta19 Dec 3, 2024

lqiu96 commented Dec 3, 2024

rmehta19 commented Dec 3, 2024

blakeli0 Dec 13, 2024 •

edited

Loading

rmehta19 Dec 13, 2024

blakeli0 Dec 13, 2024

rmehta19 Dec 13, 2024

lqiu96 Dec 19, 2024

rmehta19 Dec 21, 2024

lqiu96 Dec 19, 2024

rmehta19 Dec 21, 2024

lqiu96 Dec 19, 2024

rmehta19 Dec 21, 2024

lqiu96 Dec 19, 2024 •

edited

Loading

rmehta19 Dec 21, 2024

rmehta19 commented Jan 7, 2025

lqiu96 left a comment

rmehta19 commented Jan 9, 2025

lqiu96 Jan 9, 2025

rmehta19 Jan 9, 2025

lqiu96 commented Jan 9, 2025

rmehta19 commented Jan 9, 2025

lqiu96 Jan 9, 2025

rmehta19 Jan 9, 2025

		private String transport = "";
		private String bindingEnforcement = "";

feat: Support transport and binding-enforcement MDS parameters. #1558

feat: Support transport and binding-enforcement MDS parameters. #1558

Conversation

rmehta19 commented Oct 26, 2024

rmehta19 commented Oct 26, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

rmehta19 commented Nov 7, 2024

sonarqubecloud bot commented Nov 19, 2024

Quality Gate passed

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

lqiu96 commented Dec 3, 2024

rmehta19 commented Dec 3, 2024

blakeli0 Dec 13, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

lqiu96 Dec 19, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

rmehta19 commented Jan 7, 2025

lqiu96 left a comment

Choose a reason for hiding this comment

rmehta19 commented Jan 9, 2025

Choose a reason for hiding this comment

Choose a reason for hiding this comment

lqiu96 commented Jan 9, 2025

rmehta19 commented Jan 9, 2025

Choose a reason for hiding this comment

Choose a reason for hiding this comment

blakeli0 Dec 13, 2024 •

edited

Loading

lqiu96 Dec 19, 2024 •

edited

Loading