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

PDI-1813: Export block HCL generation for PingFederate #127

Merged
merged 10 commits into from
Aug 16, 2024

Conversation

erikostien-pingidentity
Copy link
Contributor

@erikostien-pingidentity erikostien-pingidentity commented Aug 3, 2024

  • Update Option Objects variable names to contain the service name to
    distinguish between PingOne options and PingFederate Options.
  • Update Cobra Subcommand platform export to support the new export
    service PingFederate. Add additional examples for PingFederate export
    parameters. Break out cobra flag configuration by service, and add
    PingFederate flags. Mark each PingFederate Auth method as mutually
    exclusive for the user to avoid authentication confusion.
  • Update cobra root command root.go to fail if user provided config file
    does not exist.
  • Add profile config enum type ENUM_STRING_SLICE for PF scopes and
    PF pem files. Support these slices in set_internal.go and validate.go.
  • Update export_internal.go to initialize both PingOne go client and
    PingFederate go client when needed. Update getExportableConnectors
    to use instance variables depending on the service connector specified.
  • Break out SDKClientInfo into PingOneClientInfo and PingFederateClientInfo
    for different services.
  • Create pingfederate_connector.go to export PF API resources.
  • Add first PF resource for export, pingfederate_authentication_api_application.go
  • Add server-profiles directory to hold PingFederate server-profile for
    container starting and testing.

- Update makefile and github workflows to support PingFederate testing.
- Update many variables and functions to differentiate PF services versus
PingOne services.
- Update platform export subcommand to support PingFederate export,
with Basic Auth, Access Token Auth, and Client Credentials Auth supported.
Configure the command flags to be mutually exclusive per auth method.
- Update many test cases and testing utilities to test this new functionality.
- Add PingFederate connector and one resource AuthenticationApiApplication to
test new export functionality.
- Add server-profiles folder to hold PingFederate configuration used
when starting test PingFederate containers.
@erikostien-pingidentity erikostien-pingidentity requested a review from a team as a code owner August 3, 2024 00:09
- Update Option Objects variable names to contain the service name to
distinguish between PingOne options and PingFederate Options.
- Update Cobra Subcommand `platform export` to support the new export
service PingFederate. Add additional examples for PingFederate export
parameters. Break out cobra flag configuration by service, and add
PingFederate flags. Mark each PingFederate Auth method as mutually
exclusive for the user to avoid authentication confusion.
- Update cobra root command root.go to fail if user provided config file
does not exist.
- Add profile config enum type ENUM_STRING_SLICE for PF scopes and
PF pem files. Support these slices in set_internal.go and validate.go.
- Update export_internal.go to initialize both PingOne go client and
PingFederate go client when needed. Update getExportableConnectors
to use instance variables depending on the service connector specified.
- Break out SDKClientInfo into PingOneClientInfo and PingFederateClientInfo
for different services.
- Create pingfederate_connector.go to export PF API resources.
- Add first PF resource for export, pingfederate_authentication_api_application.go
- Add server-profiles directory to hold PingFederate server-profile for
container starting and testing.
cmd/platform/export.go Outdated Show resolved Hide resolved
internal/commands/platform/export_internal_test.go Outdated Show resolved Hide resolved
@erikostien-pingidentity erikostien-pingidentity merged commit 657656e into go-cli Aug 16, 2024
8 checks passed
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