Skip to content

fix: update resource name handling and identifier parsing for emails/login_name for Users#197

Open
ike-lacey wants to merge 2 commits into
Titan-Systems:mainfrom
ike-lacey:main
Open

fix: update resource name handling and identifier parsing for emails/login_name for Users#197
ike-lacey wants to merge 2 commits into
Titan-Systems:mainfrom
ike-lacey:main

Conversation

@ike-lacey
Copy link
Copy Markdown

@ike-lacey ike-lacey commented Feb 26, 2025

SSO/Federated Logins use login_name as lookup for validation of authentication:

https://docs.snowflake.com/en/user-guide/admin-security-fed-auth-configure-idp#okta-setup

Today Titan double quotes login name on create statement.

urn::XXXXXXX:user/IKETESTLACEY {
  + name                    = "IKETESTLACEY"
  + owner                   = "USERADMIN"
  + password                = None
  + login_name              = ""IKETEST.LACEY@OLO.COM""
  + display_name            = "IKETESTLACEY"
  + first_name              = None
  + middle_name             = None
  + last_name               = None
  + email                   = None
  + must_change_password    = False
  + disabled                = False
  + days_to_expiry          = None
  + mins_to_unlock          = None
  + default_warehouse       = None
  + default_namespace       = None
  + default_role            = None
  + default_secondary_roles = None
  + mins_to_bypass_mfa      = None
  + rsa_public_key          = None
  + rsa_public_key_2        = None
  + comment                 = None
  + network_policy          = None
  + type                    = "NULL"
}

This change would reflect the changes below:

urn::XXXXXXX:user/IKETESTLACEY {
  + name                    = "IKETESTLACEY"
  + owner                   = "USERADMIN"
  + password                = None
  + login_name              = "IKETEST.LACEY@OLO.COM"
  + display_name            = "IKETESTLACEY"
  + first_name              = None
  + middle_name             = None
  + last_name               = None
  + email                   = None
  + must_change_password    = False
  + disabled                = False
  + days_to_expiry          = None
  + mins_to_unlock          = None
  + default_warehouse       = None
  + default_namespace       = None
  + default_role            = None
  + default_secondary_roles = None
  + mins_to_bypass_mfa      = None
  + rsa_public_key          = None
  + rsa_public_key_2        = None
  + comment                 = None
  + network_policy          = None
  + type                    = "NULL"
}

@ike-lacey
Copy link
Copy Markdown
Author

Hey @teej anything I can do to help move this along if possible? We're hype to move over to Titan and contribute.

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.

1 participant