pulumi-auth0 icon indicating copy to clipboard operation
pulumi-auth0 copied to clipboard

auth0.Connection constructor `strategy_version` appears to be ignored

Open ryanrasti opened this issue 2 years ago • 3 comments

What happened?

The constructor argument of auth0.Connection strategy_version and corresponding Output appear to be ignored.

It does, however, work when using the options constructor arg/output.

Steps to reproduce

Create a connection like so:

    microsoft = auth0.Connection(
        'microsoft',
        enabled_clients=[client.id],
        strategy='windowslive',
        strategy_version=2
    )

does NOT create an Auth0 connection with strategy version 2 (it uses default strategy version 1). Similiarily, looking up an existing connection

auth0.Connection.get('msft', id=...)

does not return a strategy_version in its Outputs.

Expected Behavior

Auth0 connection created with strategy_version set to 2

Actual Behavior

Auth0 connection created with (default?) strategy_version set to 1

Versions used

CLI
Version 3.35.2 Go Version go1.17.11 Go Compiler gc

Host
OS debian Version bookworm/sid Arch x86_64

Auth0 version: pulumi_auth0-2.11.0

Additional context

To actually set strategy version, one must use options. (i.e., the following WORKS):

    microsoft = auth0.Connection(
        'microsoft2',
        enabled_clients=[client.id],
        strategy='windowslive',
        options=auth0.ConnectionOptionsArgs(
            strategy_version=2
        ),
    )

Contributing

Vote on this issue by adding a 👍 reaction. To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

ryanrasti avatar Aug 02 '22 18:08 ryanrasti

Hi @ryanrasti

I apologise for the issue here - this is due to an old version of the schema in terraform that is migrating the state of the property. This should be omitted - I will get a new version of the provider upgraded and taken care of

Paul

stack72 avatar Aug 03 '22 11:08 stack72

Hi @ryanrasti - can you try the latest version of this provider and see if the issue persists? Thank you for your patience. 🙇‍♀️

guineveresaenger avatar Oct 27 '22 15:10 guineveresaenger

@guineveresaenger

Hi! I am using typescript @pulumi/auth0@^3.3.10 and when I import a Connection with options.strategyVersion: 2, it gets ignored and imported with strategyVersion 0. So it does not seem to have worked.

davidmdm avatar Aug 13 '24 20:08 davidmdm

It looks like upstream just merged a fix in this area: https://github.com/auth0/terraform-provider-auth0/pull/1024. As soon as they make a release, we'll update the pulumi provider.

iwahbe avatar Sep 27 '24 10:09 iwahbe