Skip to content

End to End TLS SSL step #2#16708

Merged
chenrujun merged 12 commits into
Azure:masterfrom
mnriem:end-to-end-tls-ssl-2
Oct 26, 2020
Merged

End to End TLS SSL step #2#16708
chenrujun merged 12 commits into
Azure:masterfrom
mnriem:end-to-end-tls-ssl-2

Conversation

@mnriem

@mnriem mnriem commented Oct 22, 2020

Copy link
Copy Markdown
Contributor

No description provided.

@ghost ghost added KeyVault azure-spring All azure-spring related issues labels Oct 22, 2020
Comment thread sdk/keyvault/azure-security-keyvault-jca/README.md
Comment thread sdk/keyvault/azure-security-keyvault-jca/pom.xml
@chenrujun

Copy link
Copy Markdown

Hi, @mnriem ,
please update your PR according to the comments in this PR: #16452

In that page, click Resolve conversation one by one after you finished related modification.

# Contributing

# Troubleshooting
## Testing the version under development

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This section should be somewhere other than the bottom to have more visibility. What about as a part of "Getting Started"?



# Azure KeyVault Certificates client library for Java
# Azure Key Vault Certificates client library for Java

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should not include these sections just so that the CI pipelines don't complain. I think that, for the sake of consistency, we should make it so that this README is in line with the one for the Secrets Spring Boot Starter. We can always include the information above in a section where it makes sense, even additional to the ones below if you think that's better.

@chenrujun chenrujun left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM.

@chenrujun chenrujun merged commit 853a45f into Azure:master Oct 26, 2020
@mnriem mnriem deleted the end-to-end-tls-ssl-2 branch October 26, 2020 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

azure-spring All azure-spring related issues KeyVault

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants