Skip to content
This repository has been archived by the owner on Sep 9, 2020. It is now read-only.

[PRED-2150] fix verify_ssl argument in api token retrieve #146

Merged
merged 7 commits into from
Dec 18, 2018

Conversation

ajbeairsto
Copy link
Contributor

@ajbeairsto ajbeairsto commented Dec 17, 2018

BATCH SCORING PULL REQUEST

This is a pull request into a public repository for Batch Scoring script maintained by DataRobot.

RATIONAL

We fail to use the verify argument in the request to get the user's api token. This means that if SSL is enabled, the request to get the API token will fail. This should fix that.

@devexp-slackbot
Copy link

@ajbeairsto ajbeairsto force-pushed the abeairsto_PRED-2150_fix_verify_ssl_in_token_acquire branch from aafe34e to 7a06efd Compare December 17, 2018 17:48
@pprett
Copy link
Contributor

pprett commented Dec 17, 2018

LGTM

please add your change to https://github.com/datarobot/batch-scoring/blob/master/CHANGES.rst and then lets release this!

pprett
pprett previously approved these changes Dec 17, 2018
@pprett
Copy link
Contributor

pprett commented Dec 17, 2018

@ajbeairsto
Copy link
Contributor Author

Thanks @pprett!

pprett
pprett previously approved these changes Dec 17, 2018
Copy link
Contributor

@pprett pprett left a comment

Choose a reason for hiding this comment

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

LGTM

@coveralls
Copy link

Coverage Status

Coverage remained the same at 83.902% when pulling f903934 on abeairsto_PRED-2150_fix_verify_ssl_in_token_acquire into 888a2c5 on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 83.902% when pulling f903934 on abeairsto_PRED-2150_fix_verify_ssl_in_token_acquire into 888a2c5 on master.

@coveralls
Copy link

coveralls commented Dec 17, 2018

Coverage Status

Coverage increased (+0.05%) to 83.949% when pulling f05280c on abeairsto_PRED-2150_fix_verify_ssl_in_token_acquire into 888a2c5 on master.

Axik
Axik previously approved these changes Dec 17, 2018
CHANGES.rst Outdated
1.15.2 (2018 Dec 17)
=====================

Enhancements
Copy link
Contributor

Choose a reason for hiding this comment

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

This is bugfix, not an enhancement :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed!

@Axik Axik merged commit c17ca96 into master Dec 18, 2018
@Axik Axik deleted the abeairsto_PRED-2150_fix_verify_ssl_in_token_acquire branch December 18, 2018 10:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants