Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix: add tls-verify parameters to request retry #118

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

fix: add tls-verify parameters to request retry #118

wants to merge 2 commits into from

Conversation

vsoch
Copy link
Contributor

@vsoch vsoch commented Nov 17, 2023 •
edited
Loading

Problem: the request retry after auth is added does not use the self._tls_verify variable.
Solution: add the variable.

This will close #117

@vsoch vsoch requested a review from SteveLasker as a code owner November 17, 2023 18:30
@vsoch vsoch mentioned this pull request Nov 17, 2023
Open
@vsoch
fix: add tls-verify parameters to request retry
10579ba 
Problem: the request retry after auth is added does not
use the self._tls_verify variable.
Solution: add the variable.

Signed-off-by: vsoch <vsoch@users.noreply.github.com>
@vsoch vsoch force-pushed the add/tls-verify-retry branch from 52c75c5 to 10579ba Compare November 17, 2023 18:43
@vsoch
update final call and remove mypy
d04367d 
Signed-off-by: vsoch <vsoch@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@SteveLasker SteveLasker Awaiting requested review from SteveLasker SteveLasker is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Missing verify argument in providers.py
1 participant
@vsoch