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

Discrepancy in Lambda Types Between Terraform Provider and FusionAuth UI/API #295

Open
sanjesh1000 opened this issue Oct 21, 2024 · 5 comments

Comments

@sanjesh1000
Copy link

I hope this message finds you well. I am currently working on managing FusionAuth using Terraform, and I have identified a couple of discrepancies that I would like to bring to your attention:

  1. Discrepancy in Available Lambda Types
    In the FusionAuth UI and API documentation, I have observed that there are 25 Lambda types available. However, when using the Terraform provider, only 20 Lambda types are supported. The Terraform provider only includes the following types:

"JWTPopulate"
"OpenIDReconcile"
"SAMLv2Reconcile"
"SAMLv2Populate"
"AppleReconcile"
"ExternalJWTReconcile"
"FacebookReconcile"
"GoogleReconcile"
"HYPRReconcile"
"TwitterReconcile"
"LDAPConnectorReconcile"
"LinkedInReconcile"
"EpicGamesReconcile"
"NintendoReconcile"
"SonyPSNReconcile"
"SteamReconcile"
"TwitchReconcile"
"XboxReconcile"
"SelfServiceRegistrationValidation"
"ClientCredentialsJWTPopulate"
This list is missing 5 Lambda types compared to what is available in the FusionAuth UI and API. Could you confirm if this is an issue with the Terraform provider, or if there are plans to include the missing Lambda types in future updates?

** Naming Style Inconsistency**
Additionally, I noticed that there is a difference in the naming style of Lambda types between the FusionAuth UI/API and the Terraform provider. In Terraform, the names are more "snake_case" (e.g., "JWTPopulate"), whereas in the FusionAuth UI/API, they follow a "Title Case" or different style.

This difference in naming convention makes it difficult to maintain consistency between the two environments, especially when working with automation tools. Could you clarify if the naming convention difference is intentional? If not, is there a possibility to standardize the naming styles between the UI/API and Terraform?

@mrudatsprint
Copy link

The missing Lambda types are:

  • SCIMServerGroupRequestConverter
  • SCIMServerGroupResponseConverter
  • SCIMServerUserRequestConverter
  • SCIMServerUserResponseConverter
  • UserInfoPopulate
  • LoginValidation

@mrudatsprint
Copy link

@sanjesh1000 The naming convention follows the format of the type request parameter.

@daethnir
Copy link

daethnir commented Dec 7, 2024

#303 would fix LoginValidation.

@daethnir
Copy link

daethnir commented Dec 8, 2024

If my fix in #303 works then I'll happily add the others, but I'd need verification it's sufficient first.

@sareh
Copy link

sareh commented Dec 13, 2024

I'm also interested in this fix. Thanks for the PR, @daethnir. I have opened a support request with the FusionAuth team regarding this discrepancy, so hopefully someone should review this ticket and your PR soon.

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

No branches or pull requests

4 participants