Skip to content

Conversation

@wesleymccollam
Copy link
Contributor

Change Description

Introduces pingcli login supported authentication. The provider will use a stored token to authenticate with the provider.

Change Characteristics

  • This PR contains beta functionality
  • This PR requires introduction of breaking changes
  • No changelog entry is needed

Checklist

All full (or complete) PRs that need review prior to merge should have the following box checked.

If contributing a partial or incomplete change (expecting the development team to complete the remaining work) please leave the box unchecked

  • Check to confirm: I have performed a review of my PR against the PR checklist and confirm that:
    • The changelog entry has been included according to the changelog process
    • Changes have proper test coverage (including regression tests)
    • Impacted resource, data source and schema descriptions have been reviewed and updated
    • Impacted resource and data source documentation HCL examples have been reviewed and updated
    • Does not introduce breaking changes (unless required to do so)
    • I am aware that changes to generated code may not be merged

Required SDK Upgrades

  • github.com/pingidentity/pingone-go-client v0.4.1

Testing

This PR has been tested with:

  • Unit tests (please paste commands and results below)
  • Acceptance tests (please paste commands and results below)
  • End-to-end tests (please paste the link to the actions workflow runs)
  • Not applicable (no evidences needed)

Shell Command(s)

go test ./internal/pingcli -v -count=1

Testing Results

Expand Results
=== RUN   TestNewConfig_EmptyPath_Error
--- PASS: TestNewConfig_EmptyPath_Error (0.00s)
=== RUN   TestNewConfig_ExpandTilde_LoadsFileAndActiveProfile
--- PASS: TestNewConfig_ExpandTilde_LoadsFileAndActiveProfile (0.00s)
=== RUN   TestGetActiveProfile_LegacyKey
--- PASS: TestGetActiveProfile_LegacyKey (0.00s)
=== RUN   TestGetProfileConfig_ByType_Success
=== RUN   TestGetProfileConfig_ByType_Success/client_credentials
=== RUN   TestGetProfileConfig_ByType_Success/authorization_code
=== RUN   TestGetProfileConfig_ByType_Success/device_code
=== RUN   TestGetProfileConfig_ByType_Success/worker
--- PASS: TestGetProfileConfig_ByType_Success (0.00s)
    --- PASS: TestGetProfileConfig_ByType_Success/client_credentials (0.00s)
    --- PASS: TestGetProfileConfig_ByType_Success/authorization_code (0.00s)
    --- PASS: TestGetProfileConfig_ByType_Success/device_code (0.00s)
    --- PASS: TestGetProfileConfig_ByType_Success/worker (0.00s)
=== RUN   TestIsFileStorageEnabled
--- PASS: TestIsFileStorageEnabled (0.00s)
=== RUN   TestLoadProfileConfig_UsesActiveProfileWhenEmpty
--- PASS: TestLoadProfileConfig_UsesActiveProfileWhenEmpty (0.00s)
=== RUN   TestLoadStoredToken_FileFallback_Success
--- PASS: TestLoadStoredToken_FileFallback_Success (0.00s)
=== RUN   TestLoadStoredToken_NoValidToken_Error
--- PASS: TestLoadStoredToken_NoValidToken_Error (0.00s)
PASS
ok  	github.com/pingidentity/terraform-provider-pingone/internal/pingcli	0.237s

@wesleymccollam wesleymccollam requested a review from a team as a code owner December 18, 2025 15:57
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