Skip to content

Conversation

@AkshatG6
Copy link
Contributor

@AkshatG6 AkshatG6 commented Oct 7, 2025

This PR addresses a runtime crash that occurs when the dictionary returned by additionalTokenRefreshParametersForAuthSession: contains non-string values, leading to a type mismatch in GTMAppAuth. (Fixes #505)

This fix is implemented in GIDEMMSupport by iterating through the additionalParameters dictionary and converts values to strings with the following logic :-

  • NSString values: Preserves them as-is.
  • NSNumber values: Converts booleans to their string representation ("true" or "false") and all other numbers to their standard string value.
  • Additionally, the return type of updatedEMMParametersWithParameters and parametersWithParameters is updated from (NSDictionary *) to (NSDictionary<NSString *, NSString *> *).

This ensures that the dictionary returned to GTMAppAuth is always of the correct type.

@AkshatG6 AkshatG6 requested a review from mdmathias October 7, 2025 22:44
@AkshatG6 AkshatG6 requested a review from mdmathias October 9, 2025 16:08
@AkshatG6 AkshatG6 merged commit e883939 into gandhiakshat/sbc-auth_time Oct 16, 2025
17 of 18 checks passed
@AkshatG6 AkshatG6 deleted the gandhiakshat/refresh-params-mismatch-from-auth_time branch October 16, 2025 20:02
@AkshatG6 AkshatG6 restored the gandhiakshat/refresh-params-mismatch-from-auth_time branch November 7, 2025 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants