Skip to content

Commit 4b8844e

Browse files
committed
fix type annotations
1 parent 71c7525 commit 4b8844e

1 file changed

Lines changed: 8 additions & 5 deletions

File tree

flask_appbuilder/security/manager.py

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
import importlib
55
import logging
66
import re
7-
from typing import Any, Callable, Dict, List, Optional, Set, Tuple, Union
7+
from typing import Any, Callable, Dict, List, Optional, Set, Tuple, TYPE_CHECKING, Union
88

99
from flask import current_app, Flask, g, request, session, url_for
1010
from flask_appbuilder.exceptions import InvalidLoginAttempt, OAuthProviderUnknown
@@ -62,6 +62,9 @@
6262
PERMISSION_PREFIX,
6363
)
6464

65+
if TYPE_CHECKING:
66+
from flask_appbuilder.security.saml.types import SAMLConfig, SAMLProvider
67+
6568
log = logging.getLogger(__name__)
6669

6770

@@ -528,21 +531,21 @@ def oauth_providers(self):
528531
return current_app.config["OAUTH_PROVIDERS"]
529532

530533
@property
531-
def saml_providers(self) -> List[Dict]:
534+
def saml_providers(self) -> List["SAMLProvider"]:
532535
return current_app.config.get("SAML_PROVIDERS", [])
533536

534537
@property
535-
def saml_config(self) -> Dict:
538+
def saml_config(self) -> "SAMLConfig":
536539
return current_app.config.get("SAML_CONFIG", {})
537540

538-
def get_saml_provider(self, name: str) -> Optional[Dict]:
541+
def get_saml_provider(self, name: str) -> Optional["SAMLProvider"]:
539542
"""Return a specific SAML provider by name."""
540543
for provider in self.saml_providers:
541544
if provider["name"] == name:
542545
return provider
543546
return None
544547

545-
def get_saml_settings(self, provider_name: str) -> Dict:
548+
def get_saml_settings(self, provider_name: str) -> "SAMLConfig":
546549
"""Build the python3-saml settings dict for a given provider.
547550
548551
Merges the global SAML_CONFIG with the provider-specific IdP config.

0 commit comments

Comments
 (0)