Skip to content

Feat: Add check for certificate auth before running API for getting user from cert - #325

Merged
mharista merged 1 commit into
developfrom
issue324
Jul 27, 2026
Merged

mharista merged 1 commit into
developfrom
issue324

Conversation

@mharista

Copy link
Copy Markdown
Contributor

Type of Change

  • New feature
  • Bug fix
  • Documentation update
  • Refactoring
  • Hotfix
  • Security patch
  • UI/UX improvement
  • Version Bump

Closes

Closes #324

Testing

Tests passing.

Effort required on reviewer's end

  • Easy
  • Medium
  • Hard

Checklist

  • I have performed a self-review of my own code

@vaibhavshaw-star vaibhavshaw-star left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Comment thread cvprac/cvp_api.py
@mharista
mharista requested a review from chetryan July 17, 2026 17:20
@mharista
mharista merged commit c3accba into develop Jul 27, 2026
3 checks passed
@mharista
mharista deleted the issue324 branch July 27, 2026 10:19
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.

Check for cert authentication before allowing use of get_cert_login_info

3 participants