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 client auth E2E test #2979

Merged
merged 249 commits into from
Apr 29, 2024
Merged

Add client auth E2E test #2979

merged 249 commits into from
Apr 29, 2024

Conversation

danielnugraha
Copy link
Contributor

@danielnugraha danielnugraha commented Feb 19, 2024

Issue

Description

There is no e2e test for client authentication feature that can make sure client authentication works end-to-end from user perspective.

Related issues/PRs

Proposal

Explanation

Add e2e test using pytorch framework.

Checklist

  • Implement proposed change
  • Write tests
  • Update documentation
  • Update the changelog entry below
  • Make CI checks pass
  • Ping maintainers on Slack (channel #contributions)

Changelog entry

Any other comments?

e2e/bare-client-auth/generate.sh Outdated Show resolved Hide resolved
e2e/bare-client-auth/generate.sh Outdated Show resolved Hide resolved
@danieljanes danieljanes changed the title Add client auth e2e Add client auth E2E test Apr 29, 2024
@danieljanes danieljanes enabled auto-merge (squash) April 29, 2024 14:10
@danieljanes danieljanes merged commit 84224e6 into main Apr 29, 2024
28 checks passed
@danieljanes danieljanes deleted the add-client-auth-e2e branch April 29, 2024 14:22
mohammadnaseri pushed a commit that referenced this pull request May 2, 2024
Co-authored-by: jafermarq <javier@flower.ai>
Co-authored-by: Daniel J. Beutel <daniel@flower.ai>
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.

4 participants