Merge pull request #282 from AzureAD/acquire-token-interactive-bugfix

Fix a malfunction in yesterday's acquire_token_interactive() PR

Author: rayluo

msal/application.py

@@ -1059,9 +1059,11 @@ def acquire_token_interactive(
                 port=port or 0),
             prompt=prompt,
             login_hint=login_hint,
-            domain_hint=domain_hint,
             timeout=timeout,
-            auth_params={"claims": claims},
+            auth_params={
+                "claims": claims,
+                "domain_hint": domain_hint,
+                },
             data=dict(kwargs.pop("data", {}), claims=claims),
             headers={
                 CLIENT_REQUEST_ID: _get_new_correlation_id(),

tests/http_client.py

@@ -10,11 +10,13 @@ def __init__(self, verify=True, proxies=None, timeout=None):
         self.timeout = timeout
 
     def post(self, url, params=None, data=None, headers=None, **kwargs):
+        assert not kwargs, "Our stack shouldn't leak extra kwargs: %s" % kwargs
         return MinimalResponse(requests_resp=self.session.post(
             url, params=params, data=data, headers=headers,
             timeout=self.timeout))
 
     def get(self, url, params=None, headers=None, **kwargs):
+        assert not kwargs, "Our stack shouldn't leak extra kwargs: %s" % kwargs
         return MinimalResponse(requests_resp=self.session.get(
             url, params=params, headers=headers, timeout=self.timeout))