Syntax revision

sanic_security/authentication.py

@@ -183,7 +183,7 @@ async def fulfill_second_factor(request: Request) -> AuthenticationSession:
     """
     authentication_session = await AuthenticationSession.decode(request)
     if not authentication_session.requires_second_factor:
-        raise SecondFactorFulfilledError()
+        raise SecondFactorFulfilledError
     two_step_session = await TwoStepSession.decode(request)
     two_step_session.validate()
     await two_step_session.check_code(request.form.get("code"))

sanic_security/authorization.py

@@ -62,7 +62,7 @@ async def check_permissions(
         logger.warning(
             f"Client {get_ip(request)} attempted an unauthorized action anonymously."
         )
-        raise AnonymousError()
+        raise AnonymousError
     roles = await authentication_session.bearer.roles.filter(deleted=False).all()
     for role in roles:
         for required_permission, role_permission in zip(
@@ -104,7 +104,7 @@ async def check_roles(request: Request, *required_roles: str) -> AuthenticationS
         logger.warning(
             f"Client {get_ip(request)} attempted an unauthorized action anonymously."
         )
-        raise AnonymousError()
+        raise AnonymousError
     roles = await authentication_session.bearer.roles.filter(deleted=False).all()
     for role in roles:
         if role.name in required_roles:

sanic_security/models.py

@@ -67,7 +67,7 @@ def validate(self) -> None:
         Raises:
             SecurityError
         """
-        raise NotImplementedError()
+        raise NotImplementedError
 
     @property
     def json(self) -> dict:
@@ -89,7 +89,7 @@ def json(self):
                     }
 
         """
-        raise NotImplementedError()
+        raise NotImplementedError
 
     class Meta:
         abstract = True
@@ -148,9 +148,9 @@ def validate(self) -> None:
         if self.deleted:
             raise DeletedError("Account has been deleted.")
         elif not self.verified:
-            raise UnverifiedError()
+            raise UnverifiedError
         elif self.disabled:
-            raise DisabledError()
+            raise DisabledError
 
     async def disable(self):
         """
@@ -321,12 +321,12 @@ def validate(self) -> None:
         if self.deleted:
             raise DeletedError("Session has been deleted.")
         elif not self.active:
-            raise DeactivatedError()
+            raise DeactivatedError
         elif (
             self.expiration_date
             and datetime.datetime.now(datetime.timezone.utc) >= self.expiration_date
         ):
-            raise ExpiredError()
+            raise ExpiredError
 
     async def deactivate(self):
         """
@@ -416,7 +416,7 @@ async def new(
         Returns:
             session
         """
-        raise NotImplementedError()
+        raise NotImplementedError
 
     @classmethod
     async def get_associated(cls, account: Account):
@@ -525,7 +525,7 @@ async def check_code(self, code: str) -> None:
                     "Your code does not match verification session code."
                 )
             else:
-                raise MaxedOutChallengeError()
+                raise MaxedOutChallengeError
         else:
             await self.deactivate()
 
@@ -621,7 +621,7 @@ def validate(self) -> None:
         """
         super().validate()
         if self.requires_second_factor:
-            raise SecondFactorRequiredError()
+            raise SecondFactorRequiredError
 
     async def refresh(self, request: Request):
         """
@@ -642,7 +642,7 @@ async def refresh(self, request: Request):
         """
         try:
             self.validate()
-            raise NotExpiredError()
+            raise NotExpiredError
         except ExpiredError as e:
             if (
                 self.refresh_expiration_date
@@ -692,7 +692,7 @@ class Role(BaseModel):
     permissions: str = fields.CharField(max_length=255, null=True)
 
     def validate(self) -> None:
-        raise NotImplementedError()
+        raise NotImplementedError
 
     @property
     def json(self) -> dict:

sanic_security/verification.py

@@ -164,7 +164,7 @@ async def verify_account(request: Request) -> TwoStepSession:
     """
     two_step_session = await TwoStepSession.decode(request)
     if two_step_session.bearer.verified:
-        raise VerifiedError()
+        raise VerifiedError
     two_step_session.validate()
     try:
         await two_step_session.check_code(request.form.get("code"))