recipe: update env vars

[gsingh91]

No description provided.

[nealrichardson]

Maybe motivate this with a use case? Like, everyone uses SOMETHING_TOKEN to store the service account key for some service, and you want to refresh that with a new value, so you need to find all content that uses that and set a new value.

[nealrichardson]

Maybe note that you need admin privileges to update across all content, but that if you're just trying to edit your content, you don't need admin privileges.

[nealrichardson]

Suggested change

	# get Server URL and API Key from the environment
	CONNECT_SERVER <- Sys.getenv('CONNECT_SERVER')
	CONNECT_API_KEY <-Sys.getenv('CONNECT_API_KEY')
	# check if API Key is valid
	if (is.null(CONNECT_API_KEY) || is.na(CONNECT_API_KEY) || nchar(CONNECT_API_KEY) == 0) {
	stop("ERROR: valid CONNECT API key is required")
	}
	# check if Server URL is valid
	if (is.null(CONNECT_SERVER) || is.na(CONNECT_SERVER) || nchar(CONNECT_SERVER) == 0) {
	stop("ERROR: valid CONNECT server URL is required")
	}

connect() will validate this for you.

[nealrichardson]

You've done library(connectapi) so you don't need to ::.

Suggested change

	client <- connectapi::connect()
	content <- connectapi::get_content(client)
	client <- connect()
	content <- get_content(client)

[nealrichardson]

It does not appear that these packages are used.

Suggested change

	library(dplyr)
	library(tidyr)
	library(glue)

[wydsith]

Suggested change

	You would need the following information in your environment to run this report:
	You need the following information in your environment to run this report:

[wydsith]

Suggested change

	Additionally, you would need to know the name of the environment variable to be updated and its new value.
	Additionally, you need to know the name of the environment variable to be updated and its new value.

[wydsith]

Suggested change

	The following script abstracts all Connect content and identifies if the content uses a specified environemnt variable, and then updates the variable with a specified value.
	The following script abstracts all Connect content, identifies if the content uses a specified environment variable, and then updates the variable with a specified value.

[kmasiello]

The default rendering of this recipe will change an environment variable for every piece of content on the server. This is irreversible and could have a big impact, especially if the target env var is something generic like "DB_PASSWORD."
I'd like the recipe to start from a place of first identifying content that has a given env var with certain filters, such as by tag or owner, and providing a "do you want to proceed y/n, this is irreversible" confirmation.