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

Add missing e2e tests for different authentication methods #1069

Open
shorim opened this issue May 13, 2024 · 0 comments
Open

Add missing e2e tests for different authentication methods #1069

shorim opened this issue May 13, 2024 · 0 comments
Labels
area/tests Writing/adding/Refactoring tests or checks kind/missing-test Categorizes issue or PR as related to missing automated tests for scenario.

Comments

@shorim
Copy link
Contributor

shorim commented May 13, 2024

Description

For MetricPipeline and TracePipeline, there are 3 different authentication methods: Mutual TLS, Basic Authentication and Token-Based With Custom Headers as doumented here and here.

For LogPipeline, we have 2 different authentication methods: Mutual TLS and Basic Authentication as documented here.

Currently we have e2e tests for the 3 pipelines only for Mutual TLS authentication. We should also add e2e tests for the other authentication methods.

Reasons

Increase our e2e test coverage.

Criterias

  • An e2e test for authentication using Basic Authentication is added for LogPipeline, MetricPipeline and TracePipeline.
  • An e2e test for authentication using Token-Based With Custom Headers is added for MetricPipeline and TracePipeline.
@shorim shorim added area/tests Writing/adding/Refactoring tests or checks kind/missing-test Categorizes issue or PR as related to missing automated tests for scenario. labels May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/tests Writing/adding/Refactoring tests or checks kind/missing-test Categorizes issue or PR as related to missing automated tests for scenario.
Projects
None yet
Development

No branches or pull requests

1 participant