Conversation
There was a problem hiding this comment.
Pull request overview
This PR implements the V1.2.1 release of the Terraform provider for FusionAuth, updating to support FusionAuth Version 1.62.0 - Refined Reindeer and fixing an issue with localized messages templates triggering changes on every execution.
Key Changes:
- Upgrades FusionAuth Go client from v1.59.0 to v1.62.0 and updates all transitive dependencies
- Adds support for new MFARequirement lambda type for multi-factor authentication requirement determination
- Adds
nameandtenant_idfields to multiple identity providers for better organization and tenant scoping - Implements form step types (collectData, verifyEmail, verifyPhoneNumber) for enhanced registration workflows
- Fixes localized messages template change detection by adding
DiffSuppressFuncto thelocalized_messagesfield
Reviewed changes
Copilot reviewed 46 out of 47 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| go.mod, go.sum | Updates FusionAuth client to v1.62.0 and dependency versions (Terraform SDK, gRPC, protobuf, etc.) |
| fusionauth/resource_fusionauth_themes.go | Adds diffSuppressTemplate to localized_messages to fix change detection issue #343 |
| fusionauth/resource_fusionauth_user_schema.go | Makes verification_ids and verification_id Optional+Computed for better API compatibility |
| fusionauth/resource_fusionauth_user.go | Implements verification_ids support in user creation/update requests |
| fusionauth/resource_fusionauth_tenant_helpers.go | Adds multi_factor_requirement_id lambda configuration support |
| fusionauth/resource_fusionauth_tenant.go | Adds multi_factor_requirement_id lambda field and reorders event types alphabetically |
| fusionauth/resource_fusionauth_lambda.go | Adds MFARequirement lambda type support |
| fusionauth/resource_fusionauth_idp_*.go | Adds name and tenant_id fields, fixes error messages, adds 404 handling for all IDP resources |
| fusionauth/resource_fusionauth_form.go | Adds step type field and makes fields Optional for verification-only steps |
| fusionauth/resource_fusionauth_application*.go | Adds multi_factor_requirement_id lambda, adds UUID validation, adds diff suppression |
| fusionauth/idphelpers.go | Adds NotFoundError constant and improved 404 handling |
| fusionauth/datasource_fusionauth_lambda.go | Adds MFARequirement type to data source |
| fusionauth/datasource_fusionauth_form.go | Adds type field to form step data source |
| fusionauth/testdata/messages.properties.go | Updates with new messages from FusionAuth 1.62.0 |
| docs/* | Updates documentation for all new fields and corrects example usage |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Collaborator
Author
|
@copilot open a new pull request to apply changes based on the comments in this thread |
|
@TomKimber I've opened a new pull request, #345, to work on those changes. Once the pull request is ready, I'll request review from you. |
7 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
V1.2.1 release