Skip to content
This repository has been archived by the owner on Mar 8, 2023. It is now read-only.

Allow grace period #19

Open
wants to merge 53 commits into
base: master
Choose a base branch
from
Open

Allow grace period #19

wants to merge 53 commits into from

Conversation

qlux
Copy link

@qlux qlux commented Aug 11, 2020

Add 2 functions to save and retrieve history of events and allow a grace period
Fixes #14

@pep8speaks
Copy link

pep8speaks commented Aug 11, 2020

Hello @qlux! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 88:80: E501 line too long (104 > 79 characters)
Line 129:80: E501 line too long (89 > 79 characters)
Line 132:25: E127 continuation line over-indented for visual indent
Line 140:34: E128 continuation line under-indented for visual indent
Line 153:5: E303 too many blank lines (2)
Line 154:28: E225 missing whitespace around operator
Line 157:21: E128 continuation line under-indented for visual indent
Line 161:21: E128 continuation line under-indented for visual indent
Line 165:17: E128 continuation line under-indented for visual indent
Line 180:33: E128 continuation line under-indented for visual indent
Line 204:80: E501 line too long (92 > 79 characters)
Line 208:25: E127 continuation line over-indented for visual indent
Line 210:17: E128 continuation line under-indented for visual indent
Line 211:17: E128 continuation line under-indented for visual indent
Line 212:17: E128 continuation line under-indented for visual indent
Line 213:17: E128 continuation line under-indented for visual indent
Line 224:17: E128 continuation line under-indented for visual indent
Line 231:25: E128 continuation line under-indented for visual indent
Line 234:25: E128 continuation line under-indented for visual indent
Line 241:17: E128 continuation line under-indented for visual indent
Line 251:25: E128 continuation line under-indented for visual indent
Line 262:45: E127 continuation line over-indented for visual indent
Line 280:80: E501 line too long (90 > 79 characters)
Line 283:80: E501 line too long (90 > 79 characters)
Line 285:80: E501 line too long (97 > 79 characters)
Line 286:80: E501 line too long (135 > 79 characters)
Line 287:80: E501 line too long (102 > 79 characters)
Line 298:80: E501 line too long (86 > 79 characters)
Line 299:30: E127 continuation line over-indented for visual indent
Line 395:80: E501 line too long (89 > 79 characters)
Line 402:80: E501 line too long (88 > 79 characters)
Line 407:13: E128 continuation line under-indented for visual indent
Line 507:17: E128 continuation line under-indented for visual indent
Line 540:21: E128 continuation line under-indented for visual indent
Line 543:25: E128 continuation line under-indented for visual indent
Line 551:80: E501 line too long (80 > 79 characters)
Line 564:80: E501 line too long (80 > 79 characters)
Line 625:26: E128 continuation line under-indented for visual indent
Line 630:80: E501 line too long (82 > 79 characters)
Line 632:30: E128 continuation line under-indented for visual indent
Line 644:80: E501 line too long (90 > 79 characters)
Line 679:80: E501 line too long (82 > 79 characters)
Line 686:80: E501 line too long (89 > 79 characters)
Line 689:80: E501 line too long (94 > 79 characters)
Line 694:1: E302 expected 2 blank lines, found 1
Line 715:80: E501 line too long (82 > 79 characters)
Line 716:26: E128 continuation line under-indented for visual indent
Line 717:26: E128 continuation line under-indented for visual indent
Line 718:26: E128 continuation line under-indented for visual indent
Line 723:19: E225 missing whitespace around operator
Line 729:80: E501 line too long (88 > 79 characters)
Line 732:29: E128 continuation line under-indented for visual indent
Line 733:29: E122 continuation line missing indentation or outdented
Line 738:80: E501 line too long (86 > 79 characters)
Line 775:80: E501 line too long (87 > 79 characters)
Line 776:21: E128 continuation line under-indented for visual indent
Line 777:21: E128 continuation line under-indented for visual indent
Line 780:39: E231 missing whitespace after ','
Line 789:1: E302 expected 2 blank lines, found 1
Line 793:80: E501 line too long (80 > 79 characters)
Line 803:1: E302 expected 2 blank lines, found 1
Line 809:1: E302 expected 2 blank lines, found 1
Line 818:80: E501 line too long (88 > 79 characters)
Line 821:80: E501 line too long (86 > 79 characters)
Line 829:21: E127 continuation line over-indented for visual indent
Line 839:1: E302 expected 2 blank lines, found 1
Line 843:41: E231 missing whitespace after ','
Line 845:1: E302 expected 2 blank lines, found 1
Line 847:80: E501 line too long (87 > 79 characters)

Line 93:20: E201 whitespace after '{'
Line 97:44: E203 whitespace before ':'
Line 99:21: E124 closing bracket does not match visual indentation
Line 100:1: E124 closing bracket does not match visual indentation
Line 132:1: E302 expected 2 blank lines, found 1
Line 134:38: E251 unexpected spaces around keyword / parameter equals
Line 134:40: E251 unexpected spaces around keyword / parameter equals
Line 155:80: E501 line too long (82 > 79 characters)
Line 168:80: E501 line too long (82 > 79 characters)
Line 172:80: E501 line too long (82 > 79 characters)
Line 274:13: E128 continuation line under-indented for visual indent
Line 325:23: E127 continuation line over-indented for visual indent

Comment last updated at 2020-11-11 19:30:54 UTC

Auto-enrollment with api key and email fixes privacyidea#2
Quentin Lux added 5 commits August 12, 2020 15:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow a grace period
2 participants