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

Store revoked state in Certificate model #1578

Merged
merged 7 commits into from
May 14, 2020

Conversation

karlerikounapuu
Copy link
Member

@karlerikounapuu karlerikounapuu commented May 5, 2020

Closes #952

When admin revokes certificate, revoked: true is applied to Certificate model. This is additionally used to check PKI validity (if apache doesn't kick the registrar beforehand).

@karlerikounapuu karlerikounapuu force-pushed the 952-revoking-cert-does-not-stop-user-from-logging-in branch 3 times, most recently from c0706cf to 3649dcb Compare May 5, 2020 11:19
@karlerikounapuu karlerikounapuu force-pushed the 952-revoking-cert-does-not-stop-user-from-logging-in branch from 3649dcb to 2dbcbf1 Compare May 5, 2020 11:23
@karlerikounapuu karlerikounapuu marked this pull request as ready for review May 5, 2020 14:04
@karlerikounapuu karlerikounapuu requested a review from yulgolem May 5, 2020 14:08
@karlerikounapuu karlerikounapuu force-pushed the 952-revoking-cert-does-not-stop-user-from-logging-in branch from 9859800 to 96b3986 Compare May 12, 2020 08:45
@karlerikounapuu karlerikounapuu force-pushed the 952-revoking-cert-does-not-stop-user-from-logging-in branch from 96b3986 to c2f8589 Compare May 12, 2020 09:29
@karlerikounapuu karlerikounapuu force-pushed the 952-revoking-cert-does-not-stop-user-from-logging-in branch from 87c4bde to 04b10ab Compare May 12, 2020 09:37
@vohmar vohmar merged commit 2730ba1 into master May 14, 2020
@vohmar vohmar deleted the 952-revoking-cert-does-not-stop-user-from-logging-in branch May 18, 2020 10:48
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.

Auth: Revoking cert does not stop user from logging in
3 participants