Skip to content

Default params to http client #192

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.

Sign up for GitHub

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

Jump to bottom
Closed
Nihisil opened this issue Mar 4, 2021 · 4 comments · Fixed by #194
Closed

Default params to http client #192

Nihisil opened this issue Mar 4, 2021 · 4 comments · Fixed by #194
Assignees
@js2me
Labels
bug Something isn't working next release

Comments

@Nihisil
Copy link

Nihisil commented Mar 4, 2021

Is there a way to set up default params for all requests? I didn't find it.

Right now I need to set {secure: true} to each request manually, which is can be improved with default params.
@js2me
Copy link
Member

js2me commented Mar 4, 2021

Hello @Nihisil !
Which kind of http client are you using ?

@Nihisil
Copy link
Author

Nihisil commented Mar 4, 2021

axios

@js2me js2me self-assigned this Mar 4, 2021
@js2me js2me added bug Something isn't working next release labels Mar 4, 2021
js2me added a commit that referenced this issue Mar 4, 2021
@js2me
fix: problem with missing default params for axios http client (issue #…
70e710c 
…192, thanks @Nihisil); docs: update CHANGELOG
@js2me js2me mentioned this issue Mar 4, 2021
Merged
js2me added a commit that referenced this issue Mar 4, 2021
@js2me
fix: problem with missing default params for axios http client (issue #…
e40d36f 
…192, thanks @Nihisil); docs: update CHANGELOG
@js2me js2me linked a pull request Mar 4, 2021 that will close this issue
Release 6.2.1 #194
Merged
@js2me js2me closed this as completed in #194 Mar 4, 2021
js2me added a commit that referenced this issue Mar 4, 2021
@js2me @artsaban
Release 6.2.1 (#194)
c9753aa 
* fix: missing generateUnionEnums in .d.ts (#193)

* fix: problem with missing default params for axios http client (issue #192, thanks @Nihisil); docs: update CHANGELOG

* bump: up version to 6.2.1

Co-authored-by: Artem Shabanov <[email protected]>
@js2me
Copy link
Member

js2me commented Mar 4, 2021

@Nihisil thanks for your bug reports!
This bug should be fixed in 6.2.1 release.
Please reopen issue if you still have the same problem.

@Nihisil
Copy link
Author

Nihisil commented Mar 4, 2021

Thank you for the quick fix!

@dkamyshov dkamyshov mentioned this issue Mar 26, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@js2me js2me

Labels
bug Something isn't working next release
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Release 6.2.1 acacode/swagger-typescript-api
2 participants
@Nihisil @js2me