Support missing token parameters on vault_okta_auth_backend resource

[vinay-gopalan]

Description

Supports missing common token parameters.

Relates OR Closes #1353

Checklist

• Added CHANGELOG entry (only for user-facing changes)
• Acceptance tests where run against all supported Vault Versions

Output from acceptance testing:

$ make testacc TESTARGS='-run=TestAccOktaAuthBackend_TokenFields'
=== RUN   TestAccOktaAuthBackend_TokenFields
=== PAUSE TestAccOktaAuthBackend_TokenFields
=== CONT  TestAccOktaAuthBackend_TokenFields
--- PASS: TestAccOktaAuthBackend_TokenFields (1.51s)
PASS

[vinay-gopalan]

This is a legacy style of checking parameters that introduced a panic. Updated to use the newer style of using resource.TestCheckAttr

[vinay-gopalan]

These were previously supported in lieu of the token_ttl and token_max_ttl fields. Since we are adding parity for those fields, these can now be deprecated.

[Zlaticanin]

This all looks good to me! Great job! 👏
I had one question, just to verify - Were these parameters added later on to Okta? Do we need to guard them with a version check, or are these available in most versions?

[vinay-gopalan]

This all looks good to me! Great job! 👏 I had one question, just to verify - Were these parameters added later on to Okta? Do we need to guard them with a version check, or are these available in most versions?

Good question! Looks like these fields have existed since before Vault 1.10.x and they just hadn't been included in the resource 😬 So we should be good, thanks for confirming 🙏🏼