Skip to content

Conversation

runwuf
Copy link

@runwuf runwuf commented Sep 12, 2022

Upgrade pymysql to 1.0.2 is required.

@runwuf runwuf force-pushed the fwu-client-cert-auth branch from 670e0aa to 1a3f4a9 Compare September 15, 2022 01:31
@runwuf runwuf requested a review from alexole September 15, 2022 01:41
Copy link
Contributor

@alexole alexole left a comment

Choose a reason for hiding this comment

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

Left a couple of comments and also, please cover new functionality with tests,

@runwuf runwuf force-pushed the fwu-client-cert-auth branch from 1a3f4a9 to 2cde254 Compare October 6, 2022 00:51
@runwuf runwuf requested a review from alexole October 6, 2022 00:53
@runwuf runwuf dismissed alexole’s stale review October 14, 2022 21:15

Changes have been done, but tests outstanding. A JIRA request has been submitted if BF team could finish up the rest including avn cli. Thank you.

Copy link

@paulwouters paulwouters left a comment

Choose a reason for hiding this comment

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

LGTM

Comment on lines +77 to +79
* `SOURCE_SSL_CA` - The path name of the Certificate Authority (CA) certificate file in PEM format.
* `SOURCE_SSL_CERT` - The path name of the server SSL public key certificate file in PEM format.
* `SOURCE_SSL_KEY` - The path name of the server SSL private key file in PEM format.

Choose a reason for hiding this comment

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

should these be SOURCE_SERVICE_SSL_XXX to match the other keywords better?

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.

3 participants