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

Add config options of the list type #31

Merged
merged 1 commit into from
May 3, 2023
Merged

Conversation

tombruijn
Copy link
Member

@tombruijn tombruijn commented May 2, 2023

Parse lists from environment variables, and format them again when set as private env vars for the agent.

I left out request_headers, because we should check what the default for that should be in more detail. This is more the general structure of parsing and setting config options that the client does not directly use (right now).

[skip changeset]
Part of #16
Based on #28

src/appsignal/config.py Outdated Show resolved Hide resolved
@tombruijn tombruijn force-pushed the boolean-config-options branch 3 times, most recently from dafe175 to 44210ce Compare May 3, 2023 07:48
Parse lists from environment variables, and format them again when set
as private env vars for the agent.

I left out request_headers, because we should check what the default for
that should be in more detail. This is more the general structure of
parsing and setting config options that the client does not directly use
(right now).
@tombruijn tombruijn changed the base branch from boolean-config-options to main May 3, 2023 07:50
@tombruijn tombruijn merged commit f591144 into main May 3, 2023
1 check passed
@tombruijn tombruijn deleted the list-config-options branch May 3, 2023 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants