Auth.run() - Code Metrics - Inspection of "WIP: Add duo pack (2FA)" - StackStorm/st2contrib - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#442)

unknown

created 2016-03-14 17:25 UTC

Auth.run() F

↳ Parent: Auth

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Metric	Value
cc	11
dl	0
loc	69
rs	3.6486

How to fix Long Method Complexity

#!/usr/bin/env python

# Licensed to the StackStorm, Inc ('StackStorm') under one or more
# contributor license agreements.  See the NOTICE file distributed with
# this work for additional information regarding copyright ownership.
# The ASF licenses this file to You under the Apache License, Version 2.0
# (the "License"); you may not use this file except in compliance with
# the License.  You may obtain a copy of the License at
#
#     http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.

import urllib

from st2actions.runners.pythonrunner import Action
import duo_client


class Auth(Action):
    def run(self, username, factor,
            ipaddr, device, push_type, passcode, pushinfo):
        """
        Auth against the Duo Platorm.

        Returns: An dict with info returned by Duo.

        Raises:
          ValueError: 'Duo config not found in config' or 'Invalid factor'
          RuntimeError: 'Failed auth.'
        """

        try:
            ikey = self.config['auth']['ikey']
            skey = self.config['auth']['skey']
            host = self.config['auth']['host']
        except KeyError:
            raise ValueError("Duo config not found in config.")

        auth = duo_client.Auth(ikey=ikey,
                               skey=skey,
                               host=host)

        auth_kargs = {}

        if factor == "auto" or factor == "push":
            auth_kargs['type'] = push_type
            auth_kargs['device'] = device

            if ipaddr is not None:
                auth_kargs['ipaddr'] = ipaddr

            if pushinfo is not None:
                info = {}
                for value in pushinfo.split('; '):
                    (key, value) = value.split('=')
                    info[key] = value

                encoded = urllib.urlencode(info)
                auth_kargs['pushinfo'] = encoded
        elif factor == "passcode":
            auth_kargs['passcode'] = passcode
        elif factor == "phone":
            auth_kargs['device'] = device
        elif factor == "sms":
            # As 'sms' just denies and then we do not support it
            # requires re-authentication.

            print "Denied, we do not support SMS!"
            raise ValueError("Denied, we do not support SMS!")
        else:
            raise ValueError("Invalid factor!")

        try:
            data = auth.auth(factor=factor,
                             username=username,
                             **auth_kargs)
        except RuntimeError, e:
            print "Error: %s" % e
            raise RuntimeError("Error: %s" % e)
        else:
            if data['result'] == "allow":
                return data
            elif data['result'] == "deny":
                print data['status_msg']
                raise RuntimeError("{}".format(
                    data['status_msg']))
            else:
                raise RuntimeError("Invalid status")


1			#!/usr/bin/env python
2
3			# Licensed to the StackStorm, Inc ('StackStorm') under one or more
4			# contributor license agreements. See the NOTICE file distributed with
5			# this work for additional information regarding copyright ownership.
6			# The ASF licenses this file to You under the Apache License, Version 2.0
7			# (the "License"); you may not use this file except in compliance with
8			# the License. You may obtain a copy of the License at
9			#
10			# http://www.apache.org/licenses/LICENSE-2.0
11			#
12			# Unless required by applicable law or agreed to in writing, software
13			# distributed under the License is distributed on an "AS IS" BASIS,
14			# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
15			# See the License for the specific language governing permissions and
16			# limitations under the License.
17
18			import urllib
19
20			from st2actions.runners.pythonrunner import Action
21			import duo_client
22
23
24			class Auth(Action):
25			def run(self, username, factor,
26			ipaddr, device, push_type, passcode, pushinfo):
27			"""
28			Auth against the Duo Platorm.
29
30			Returns: An dict with info returned by Duo.
31
32			Raises:
33			ValueError: 'Duo config not found in config' or 'Invalid factor'
34			RuntimeError: 'Failed auth.'
35			"""
36
37			try:
38			ikey = self.config['auth']['ikey']
39			skey = self.config['auth']['skey']
40			host = self.config['auth']['host']
41			except KeyError:
42			raise ValueError("Duo config not found in config.")
43
44			auth = duo_client.Auth(ikey=ikey,
45			skey=skey,
46			host=host)
47
48			auth_kargs = {}
49
50			if factor == "auto" or factor == "push":
51			auth_kargs['type'] = push_type
52			auth_kargs['device'] = device
53
54			if ipaddr is not None:
55			auth_kargs['ipaddr'] = ipaddr
56
57			if pushinfo is not None:
58			info = {}
59			for value in pushinfo.split('; '):
60			(key, value) = value.split('=')
61			info[key] = value
62
63			encoded = urllib.urlencode(info)
64			auth_kargs['pushinfo'] = encoded
65			elif factor == "passcode":
66			auth_kargs['passcode'] = passcode
67			elif factor == "phone":
68			auth_kargs['device'] = device
69			elif factor == "sms":
70			# As 'sms' just denies and then we do not support it
71			# requires re-authentication.
72
73			print "Denied, we do not support SMS!"
74			raise ValueError("Denied, we do not support SMS!")
75			else:
76			raise ValueError("Invalid factor!")
77
78			try:
79			data = auth.auth(factor=factor,
80			username=username,
81			**auth_kargs)
82			except RuntimeError, e:
83			print "Error: %s" % e
84			raise RuntimeError("Error: %s" % e)
85			else:
86			if data['result'] == "allow":
87			return data
88			elif data['result'] == "deny":
89			print data['status_msg']
90			raise RuntimeError("{}".format(
91			data['status_msg']))
92			else:
93			raise RuntimeError("Invalid status")
94

StackStorm / st2contrib

Pull Request — master (#442)

Auth.run() F

Complexity

Size

Duplication

How to fix Long Method Complexity

Long Method

Complexity

Duplication Side-by-Side

Filter issues like