-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Added List Certificate Function #587
Conversation
Update back to Main
Merge back to Master
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixes #563
At the moment |
For my personal Use case I need to list revoked users as well to see them. But I can see why that would not be needed. I can take remove all revoked users or potentially in a future update add another submenu to list All or only valid Certificates? |
@psgoundar I'd like to see revoked users too but there are some user who asked to not show them #566 (comment). My previous comment was to link to the issue not to request changes, @angristan will decide on this. |
openvpn-install.sh
Outdated
echo " 3) List Current Issued Certificates" | ||
echo " 8) Remove OpenVPN" | ||
echo " 9) Exit" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3, 8, 9?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've got multiple other options that I've added to my repo And I was planning on slowly submitting all the changes up to the main repo.
And I didn't want to continually be renumbering the remove and Exit options
Such as:
echo " 1) Add a new user"
echo " 2) Revoke existing user"
echo " 3) List Current Issued Certificates"
echo " 4) List Current Active Users"
echo " 5) Backup Configuration"
echo " 6) Restore Configuration from Backup *Incomplete"
echo " 7) Sync Configuration to Alternate Servers *Incomplete"
echo " 8) Remove OpenVPN"
echo " 9) Exit"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please do the appropriate changes for this PR :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I totally Screwed up my branch. I will close this pull Request. And pull another with the neccessary Changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok :)
Repulled Main Repo and Redoing changes |
This is a list function that show all the issues Certificates from the /etc/openvpn/easy-rsa/pki/index.txt file and formats it into a table to show the Status, Name and Expiration Dates.