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

control-service: Kerberos authentication IT #798

Merged
merged 23 commits into from
May 9, 2022

Conversation

mrMoZ1
Copy link
Contributor

@mrMoZ1 mrMoZ1 commented Apr 11, 2022

why: As a part of recently introduced functionality to provide kerberos authentication
we needed to add an integration test with a running control service.

what: Added Kerberos authentication tests against a running control service. The test
creates it's own Kerberos server (using apache kerby) and creates as unique keytab
used to authenticate against the server each time. The test verifies that authentication
works as expected.

tests: n/a

Signed-off-by: Momchil Zhivkov [email protected]

Signed-off-by: mrMoZ1 <[email protected]>
@mrMoZ1 mrMoZ1 changed the title control-service: Kerberos authentication IT [draft] control-service: Kerberos authentication IT Apr 11, 2022
mrMoZ1 added 9 commits April 18, 2022 15:23
Signed-off-by: mrMoZ1 <[email protected]>
Signed-off-by: mrMoZ1 <[email protected]>
Signed-off-by: mrMoZ1 <[email protected]>
Signed-off-by: mrMoZ1 <[email protected]>
Signed-off-by: mrMoZ1 <[email protected]>
Signed-off-by: mrMoZ1 <[email protected]>
Signed-off-by: mrMoZ1 <[email protected]>
Signed-off-by: mrMoZ1 <[email protected]>
@mrMoZ1 mrMoZ1 marked this pull request as ready for review April 19, 2022 14:24
@mrMoZ1 mrMoZ1 changed the title [draft] control-service: Kerberos authentication IT control-service: Kerberos authentication IT Apr 19, 2022
@mrMoZ1 mrMoZ1 force-pushed the person/mzhivkov/kerberos-integration-test branch from b35be3b to b121682 Compare April 20, 2022 08:19
Copy link
Collaborator

@antoniivanov antoniivanov left a comment

Choose a reason for hiding this comment

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

LGTM. You can write a bit more documentation if you wish and merge it.

Signed-off-by: mrMoZ1 <[email protected]>
@mrMoZ1 mrMoZ1 enabled auto-merge (squash) May 9, 2022 11:06
@mrMoZ1 mrMoZ1 merged commit 6c5a4b0 into main May 9, 2022
@mrMoZ1 mrMoZ1 deleted the person/mzhivkov/kerberos-integration-test branch May 9, 2022 11:17
mrMoZ1 added a commit that referenced this pull request May 9, 2022
mrMoZ1 added a commit that referenced this pull request May 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants