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

Set proper values for 'OP_INTEGRATION_*' env vars #124

Closed
volodymyrZotov opened this issue Dec 13, 2023 · 0 comments · Fixed by #153
Closed

Set proper values for 'OP_INTEGRATION_*' env vars #124

volodymyrZotov opened this issue Dec 13, 2023 · 0 comments · Fixed by #153
Assignees
Labels

Comments

@volodymyrZotov
Copy link
Collaborator

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.

@volodymyrZotov volodymyrZotov added the bug Something isn't working label Dec 13, 2023
@jillianwilson jillianwilson removed the bug Something isn't working label Feb 1, 2024
@edif2008 edif2008 self-assigned this Feb 27, 2024
@github-actions github-actions bot mentioned this issue Mar 12, 2024
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants