Skip to content
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.

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

Support self-signed certificate in the tests of Application Connector components #1486

Merged
merged 9 commits into from
Nov 7, 2018
Merged

Support self-signed certificate in the tests of Application Connector components #1486

merged 9 commits into from
Nov 7, 2018

Conversation

franpog859
Copy link
Contributor

@franpog859 franpog859 commented Oct 31, 2018

Description

Changes proposed in this pull request:

  • Add configuration flag skipSslVerify to skip SSL verification in the Application Connector tests - enable self-signed certificate
  • Skipping SSL verification is set to false by default on cluster. To enable self-signed certificate set the .Values.tests.skipSslVerify value to true in the connector-service-test

Related issue(s)

Resolves #1464

@franpog859 franpog859 added kind/feature Categorizes issue or PR as related to a new feature. WIP area/application-connector Issues or PRs related to application connectivity labels Oct 31, 2018
@franpog859 franpog859 added this to the Sprint_Framefrog_4 milestone Oct 31, 2018
@franpog859 franpog859 removed the WIP label Nov 2, 2018
@franpog859 franpog859 merged commit 35907be into kyma-project:master Nov 7, 2018
@franpog859 franpog859 deleted the 1464-The-tests-of-Application-Connector-components-must-support-cases-with-self-signed-certificates branch February 20, 2019 14:20
grischperl pushed a commit to grischperl/kyma that referenced this pull request Nov 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/application-connector Issues or PRs related to application connectivity kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants