Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Google IdP may not load JavaScript with application override #2186

Closed
2 tasks done
spwitt opened this issue Mar 29, 2023 · 1 comment
Closed
2 tasks done

Google IdP may not load JavaScript with application override #2186

spwitt opened this issue Mar 29, 2023 · 1 comment
Assignees
Labels
bug Something isn't working
Milestone

Comments

@spwitt
Copy link

spwitt commented Mar 29, 2023

Google IdP does not follow application override for login method

Description

If the Google IdP is configured to "Use Redirect" for the login method, and the application override is "Use Popup" or "Use JavaScript," the supporting JavaScript is not loaded.

During resolution and testing for the issue described above, other configuration issues were identified. The API and button properties for the application overrides were not being handled correctly.

The Helper.js file for IdPs should also be loaded up front rather than loaded dynamically from other JavaScript.

Affects versions

1.44.0

Steps to reproduce

  1. Set base Google IdP to "Use Redirect" for login method
  2. Add an application override that sets login method to "Use Popup"
  3. JavaScript for Google sign in is not loaded on the /oauth2/authorize page

Expected behavior

  1. The JavaScript should be loaded based on the application override.
  2. The Helper.js file should be loaded up front rather than appended to the DOM from JavaScript
  3. The application-level API and button properties should be merged with the base configuration

Workaround

In the Helpers file in your theme, find the following line:

[#if identityProviders["Google"]?has_content && identityProviders["Google"][0].loginMethod != "UseRedirect"]

and replace it with

[#if identityProviders["Google"]?has_content && identityProviders["Google"][0].lookupLoginMethod(clientId) != "UseRedirect"]

Community guidelines

All issues filed in this repository must abide by the FusionAuth community guidelines.

Additional context

Add any other context about the problem here.

Documentation

  • Override ux_mode=redirect when loginMethod is UsePopup
  • Individual property overrides for Application-specific settings

This PR covers doc tasks:

@spwitt spwitt added the bug Something isn't working label Mar 29, 2023
@spwitt spwitt self-assigned this Mar 29, 2023
@spwitt
Copy link
Author

spwitt commented Mar 30, 2023

@robotdan robotdan added this to Backlog in FusionAuth Issues via automation Apr 5, 2023
@robotdan robotdan moved this from Backlog to Code complete in FusionAuth Issues Apr 5, 2023
@robotdan robotdan added this to the 1.46.0 milestone Apr 11, 2023
@robotdan robotdan moved this from Code complete to Reviewer approved in FusionAuth Issues Apr 12, 2023
@robotdan robotdan modified the milestones: 1.46.0, 1.45.1 Apr 21, 2023
@spwitt spwitt moved this from Reviewer approved to Done in FusionAuth Issues Apr 21, 2023
@spwitt spwitt closed this as completed Apr 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
FusionAuth Issues
  
Delivered
Development

No branches or pull requests

2 participants