Skip to content

Commit 9ebf18d

Browse files
Copilotsomalaya
andauthored
fix: add @jvmoverloads to ReAttachPrtHeaderHandler and NonceRedirectHandler constructors
Kotlin default constructor parameters are not visible to Java callers without @jvmoverloads. The two 3-argument Java call sites of ReAttachPrtHeaderHandler (processCrossCloudRedirect, processWebCpAuthorize) were getting compile errors because no 3-arg overload existed. @jvmoverloads generates the necessary overloaded constructors for Java interop. Same annotation added to NonceRedirectHandler for consistency. Agent-Logs-Url: https://github.com/AzureAD/microsoft-authentication-library-common-for-android/sessions/f90300a1-6ff8-4474-b169-3d5683a0722e Co-authored-by: somalaya <69237821+somalaya@users.noreply.github.com>
1 parent d8979bd commit 9ebf18d

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

common/src/main/java/com/microsoft/identity/common/internal/ui/webview/challengehandlers/NonceRedirectHandler.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ import java.net.URL
3535
/**
3636
* Handler for processing nonce from redirect and attaching new prt credential header on web view.
3737
*/
38-
class NonceRedirectHandler(
38+
class NonceRedirectHandler @JvmOverloads constructor(
3939
private val webView: WebView,
4040
private val headers: HashMap<String, String>,
4141
private val span : Span,

common/src/main/java/com/microsoft/identity/common/internal/ui/webview/challengehandlers/ReAttachPrtHeaderHandler.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ import io.opentelemetry.api.trace.Span
3333
/**
3434
* Handler for attaching prt credential header on web view in redirect scenarios.
3535
*/
36-
class ReAttachPrtHeaderHandler(
36+
class ReAttachPrtHeaderHandler @JvmOverloads constructor(
3737
private val webView: WebView,
3838
private val headers: HashMap<String, String>,
3939
private val span : Span,

0 commit comments

Comments
 (0)