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 additional options support to base #112

Closed
wants to merge 2 commits into from

Conversation

WardsParadox
Copy link
Contributor

Add additional options field that can be specified to modify all curl requests in the event they are needed by certain environments (like an additional SSO login cookie)

Resolves #109

@grahampugh
Copy link
Owner

Hi, I'm afraid this conflicts with changes that were made since this PR to cope with API Clients. Some name other than curl_additional_opts is going to be needed, since we have additional_curl_opts already. Maybe custom_curl_opts?

@WardsParadox
Copy link
Contributor Author

Should be a fairly easy lift. We've just been using the fork for this long to get it working. I'll see if I can get that done today so we can potentially merge back to main :)

@WardsParadox
Copy link
Contributor Author

Hmm despite pulling from main and rebasing, seems like a conflict. Let me retry that

@WardsParadox WardsParadox deleted the additional_opts branch July 18, 2024 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature Request: Allow Autopkg Processors to extend curl options
2 participants