Skip to content

allow to configure auth_header_name and prefix in AuthenticatedClient #660

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

Merged

Conversation

supermihi
Copy link
Contributor

Before security scheme support has been fully implemented (cf. #600), this PR at least allows to manually configure the correct authentication header without having to use custom templates.

Note that a custom API key prefix is also supported in the "old" OpenAPI generator.

@supermihi supermihi force-pushed the configure-authenticated-client branch 2 times, most recently from 3e64751 to f5e5259 Compare August 26, 2022 12:28
@dbanty dbanty added the ✨ enhancement New feature or improvement label Aug 27, 2022
@dbanty dbanty added this to the 0.11.6 milestone Aug 27, 2022
Copy link
Collaborator

@dbanty dbanty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@codecov
Copy link

codecov bot commented Aug 27, 2022

Codecov Report

Merging #660 (2e06a48) into main (6245b18) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #660   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           49        49           
  Lines         1794      1794           
=========================================
  Hits          1794      1794           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@dbanty dbanty enabled auto-merge (squash) August 27, 2022 21:34
@dbanty dbanty disabled auto-merge August 27, 2022 21:37
@dbanty dbanty merged commit 4f9f03a into openapi-generators:main Aug 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement New feature or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants