1Password/terraform-provider-onepassword

Set proper values for 'OP_INTEGRATION_*' env vars

volodymyrZotov opened this issue · 0 comments

We need to set proper 'OP_INTEGRATION_*' env var values when using CLI.

Currently, they are:

cmd.Env = append(cmd.Environ(),
		"OP_FORMAT=json",
		"OP_INTEGRATION_NAME=terraform-provider-connect",
		"OP_INTEGRATION_ID=GO",
		//"OP_INTEGRATION_BUILDNUMBER="+version.ProviderVersion, // causes bad request errors from CLI
	)

It should be changed to:

cmd.Env = append(cmd.Environ(),
		"OP_FORMAT=json",
		"OP_INTEGRATION_NAME=terraform-provider",
		"OP_INTEGRATION_ID=TP",
		"OP_INTEGRATION_BUILDNUMBER="+version.ProviderVersion,
	)

Note:
Setting OP_INTEGRATION_BUILDNUMBER env var caused CLI error. In scope of this issue, we need to investigate how to pass to properly so the CLI throws no errors.