clarify ambiguities in destination, slightly better error messages

[nu-vpn-proxy.git] / gp-saml-gui.py

diff --git a/gp-saml-gui.py b/gp-saml-gui.py
index e4298ddd9459db3b393d261b993e7ac1e50b0042..0f2f96179308423d64b20f606668910f77c2c5f6 100755 (executable)
--- a/gp-saml-gui.py
+++ b/gp-saml-gui.py
@@ -6,16 +6,18 @@ import pprint
 import urllib
 import requests
 import xml.etree.ElementTree as ET
 import urllib
 import requests
 import xml.etree.ElementTree as ET

-import os
+import ssl

 
 

+from operator import setitem
+from os import path
+from shlex import quote

 from sys import stderr
 from binascii import a2b_base64, b2a_base64
 from sys import stderr
 from binascii import a2b_base64, b2a_base64

+from urllib.parse import urlparse

 
 gi.require_version('Gtk', '3.0')
 
 gi.require_version('Gtk', '3.0')

-from gi.repository import Gtk
-

 gi.require_version('WebKit2', '4.0')
 gi.require_version('WebKit2', '4.0')

-from gi.repository import WebKit2
+from gi.repository import Gtk, WebKit2, GLib

 
 class SAMLLoginView:
     def __init__(self, uri, html=None, verbose=False, cookies=None, verify=True):
 
 class SAMLLoginView:
     def __init__(self, uri, html=None, verbose=False, cookies=None, verify=True):


@@ -23,6 +25,7 @@ class SAMLLoginView:
 
         # API reference: https://lazka.github.io/pgi-docs/#WebKit2-4.0
 
 
         # API reference: https://lazka.github.io/pgi-docs/#WebKit2-4.0
 

+        self.closed = False

         self.success = False
         self.saml_result = {}
         self.verbose = verbose
         self.success = False
         self.saml_result = {}
         self.verbose = verbose


@@ -40,7 +43,7 @@ class SAMLLoginView:
         window.add(self.wview)
         window.show_all()
         window.set_title("SAML Login")
         window.add(self.wview)
         window.show_all()
         window.set_title("SAML Login")

-        window.connect('delete-event', Gtk.main_quit)
+        window.connect('delete-event', self.close)

         self.wview.connect('load-changed', self.get_saml_headers)
         self.wview.connect('resource-load-started', self.log_resources)
 
         self.wview.connect('load-changed', self.get_saml_headers)
         self.wview.connect('resource-load-started', self.log_resources)
 


@@ -49,44 +52,89 @@ class SAMLLoginView:
         else:
             self.wview.load_uri(uri)
 
         else:
             self.wview.load_uri(uri)
 

+    def close(self, window, event):
+        self.closed = True
+        Gtk.main_quit()
+

     def log_resources(self, webview, resource, request):
         if self.verbose > 1:
     def log_resources(self, webview, resource, request):
         if self.verbose > 1:

-            print('%s for resource %s' % (request.get_http_method() or 'Request', resource.get_uri()), file=stderr)
+            print('[REQUEST] %s for resource %s' % (request.get_http_method() or 'Request', resource.get_uri()), file=stderr)
+        if self.verbose > 2:
+            resource.connect('finished', self.log_resource_details, request)
+
+    def log_resource_details(self, resource, request):
+        m = request.get_http_method() or 'Request'
+        uri = resource.get_uri()
+        rs = resource.get_response()
+        h = rs.get_http_headers() if rs else None
+        if h:
+            ct, cl = h.get_content_type(), h.get_content_length()
+            content_type, charset = ct[0], ct.params.get('charset')
+            content_details = '%d bytes of %s%s for ' % (cl, content_type, ('; charset='+charset) if charset else '')
+        print('[RECEIVE] %sresource %s %s' % (content_details if h else '', m, uri), file=stderr)
+
+    def log_resource_text(self, resource, result, content_type, charset=None, show_headers=None):
+        data = resource.get_data_finish(result)
+        content_details = '%d bytes of %s%s for ' % (len(data), content_type, ('; charset='+charset) if charset else '')
+        print('[DATA   ] %sresource %s' % (content_details, resource.get_uri()), file=stderr)
+        if show_headers:
+            for h,v in show_headers.items():
+                print('%s: %s' % (h, v), file=stderr)
+            print(file=stderr)
+        if charset or content_type.startswith('text/'):
+            print(data.decode(charset or 'utf-8'), file=stderr)

 
     def get_saml_headers(self, webview, event):
         if event != WebKit2.LoadEvent.FINISHED:
             return
 
         mr = webview.get_main_resource()
 
     def get_saml_headers(self, webview, event):
         if event != WebKit2.LoadEvent.FINISHED:
             return
 
         mr = webview.get_main_resource()

-        if self.verbose:
-            print("Finished loading %s" % mr.get_uri(), file=stderr)
+        uri = mr.get_uri()

         rs = mr.get_response()
         h = rs.get_http_headers()
         rs = mr.get_response()
         h = rs.get_http_headers()

-        if h:
-            l = []
-            def listify(name, value, t=l):
-                if (name.startswith('saml-') or name in ('prelogin-cookie', 'portal-userauthcookie')):
-                    t.append((name, value))
-            h.foreach(listify)
-            d = dict(l)
-            if d and self.verbose:
-                print("Got SAML result headers: %r" % d, file=stderr)
-            d = self.saml_result
-            d.update(dict(l))
-            if 'saml-username' in d and ('prelogin-cookie' in d or 'portal-userauthcookie' in d):
-                print("Got all required SAML headers, done.", file=stderr)
-                self.success = True
-                Gtk.main_quit()
+        if self.verbose:
+            print('[PAGE   ] Finished loading page %s' % uri, file=stderr)
+        if not h:
+            return
+
+        # convert to normal dict
+        d = {}
+        h.foreach(lambda k, v: setitem(d, k, v))
+        # filter to interesting headers
+        fd = {name:v for name, v in d.items() if name.startswith('saml-') or name in ('prelogin-cookie', 'portal-userauthcookie')}
+        if fd and self.verbose:
+            print("[SAML   ] Got SAML result headers: %r" % fd, file=stderr)
+            if self.verbose > 1:
+                # display everything we found
+                ct = h.get_content_type()
+                mr.get_data(None, self.log_resource_text, ct[0], ct.params.get('charset'), d)
+
+        # check if we're done
+        self.saml_result.update(fd, server=urlparse(uri).netloc)
+        GLib.timeout_add(1000, self.check_done)
+
+    def check_done(self):
+        d = self.saml_result
+        if 'saml-username' in d and ('prelogin-cookie' in d or 'portal-userauthcookie' in d):
+            if args.verbose:
+                print("[SAML   ] Got all required SAML headers, done.", file=stderr)
+            self.success = True
+            Gtk.main_quit()

 
 def parse_args(args = None):
     p = argparse.ArgumentParser()
     p.add_argument('server', help='GlobalProtect server (portal or gateway)')
     p.add_argument('--no-verify', dest='verify', action='store_false', default=True, help='Ignore invalid server certificate')
 
 def parse_args(args = None):
     p = argparse.ArgumentParser()
     p.add_argument('server', help='GlobalProtect server (portal or gateway)')
     p.add_argument('--no-verify', dest='verify', action='store_false', default=True, help='Ignore invalid server certificate')

-    p.add_argument('-C', '--no-cookies', dest='cookies', action='store_const', const=None,
-                   default='~/.gp-saml-gui-cookies', help="Don't use cookies (stored in %(default)s)")

     x = p.add_mutually_exclusive_group()
     x = p.add_mutually_exclusive_group()

-    x.add_argument('-p','--portal', dest='portal', action='store_true', help='SAML auth to portal')
-    x.add_argument('-g','--gateway', dest='portal', action='store_false', help='SAML auth to gateway (default)')
+    x.add_argument('-C', '--cookies', default='~/.gp-saml-gui-cookies',
+                   help='Use and store cookies in this file (instead of default %(default)s)')
+    x.add_argument('-K', '--no-cookies', dest='cookies', action='store_const', const=None,
+                   help="Don't use or store cookies at all")
+    x = p.add_mutually_exclusive_group()
+    x.add_argument('-p','--portal', dest='interface', action='store_const', const='portal', default='gateway',
+                   help='SAML auth to portal')
+    x.add_argument('-g','--gateway', dest='interface', action='store_const', const='gateway',
+                   help='SAML auth to gateway (default)')

     g = p.add_argument_group('Client certificate')
     g.add_argument('-c','--cert', help='PEM file containing client certificate (and optionally private key)')
     g.add_argument('--key', help='PEM file containing client private key (if not included in same file as certificate)')
     g = p.add_argument_group('Client certificate')
     g.add_argument('-c','--cert', help='PEM file containing client certificate (and optionally private key)')
     g.add_argument('--key', help='PEM file containing client private key (if not included in same file as certificate)')


@@ -100,7 +148,7 @@ def parse_args(args = None):
     args.extra = dict(x.split('=', 1) for x in args.extra)
 
     if args.cookies:
     args.extra = dict(x.split('=', 1) for x in args.extra)
 
     if args.cookies:

-        args.cookies = os.path.expanduser(args.cookies)
+        args.cookies = path.expanduser(args.cookies)

 
     if args.cert and args.key:
         args.cert, args.key = (args.cert, args.key), None
 
     if args.cert and args.key:
         args.cert, args.key = (args.cert, args.key), None


@@ -120,17 +168,40 @@ if __name__ == "__main__":
     s.headers['User-Agent'] = 'PAN GlobalProtect'
     s.cert = args.cert
 
     s.headers['User-Agent'] = 'PAN GlobalProtect'
     s.cert = args.cert
 

+    if2prelogin = {'portal':'global-protect/prelogin.esp','gateway':'ssl-vpn/prelogin.esp'}
+    if2auth = {'portal':'global-protect/getconfig.esp','gateway':'ssl-vpn/login.esp'}
+

     # query prelogin.esp and parse SAML bits
     if args.uri:
         sam, uri, html = 'URI', args.server, None
     else:
     # query prelogin.esp and parse SAML bits
     if args.uri:
         sam, uri, html = 'URI', args.server, None
     else:

-        endpoint = 'https://{}/{}/prelogin.esp'.format(args.server, ('global-protect' if args.portal else 'ssl-vpn'))
-        res = s.post(endpoint, verify=args.verify, data=args.extra)
+        endpoint = 'https://{}/{}'.format(args.server, if2prelogin[args.interface])
+        if args.verbose:
+            print("Looking for SAML auth tags in response to %s..." % endpoint, file=stderr)
+        try:
+            res = s.post(endpoint, verify=args.verify, data=args.extra)
+        except Exception as ex:
+            rootex = ex
+            while True:
+                if isinstance(rootex, ssl.SSLError):
+                    break
+                elif not rootex.__cause__ and not rootex.__context__:
+                    break
+                rootex = rootex.__cause__ or rootex.__context__
+            if isinstance(rootex, ssl.CertificateError):
+                p.error("SSL certificate error (try --no-verify to ignore): %s" % rootex)
+            elif isinstance(rootex, ssl.SSLError):
+                p.error("SSL error: %s" % rootex)
+            else:
+                raise

         xml = ET.fromstring(res.content)
         xml = ET.fromstring(res.content)

+        if xml.tag != 'prelogin-response':
+            p.error("This does not appear to be a GlobalProtect prelogin response\nCheck in browser: {}".format(endpoint))

         sam = xml.find('saml-auth-method')
         sr = xml.find('saml-request')
         if sam is None or sr is None:
         sam = xml.find('saml-auth-method')
         sr = xml.find('saml-request')
         if sam is None or sr is None:

-            p.error("This does not appear to be a SAML prelogin response (<saml-auth-method> or <saml-request> tags missing)")
+            p.error("This does not appear to be a SAML prelogin response (<saml-auth-method> or <saml-request> tags missing)\n"
+                    "Check in browser: {}".format(endpoint))

         sam = sam.text
         sr = a2b_base64(sr.text).decode()
         if sam == 'POST':
         sam = sam.text
         sr = a2b_base64(sr.text).decode()
         if sam == 'POST':


@@ -154,21 +225,45 @@ if __name__ == "__main__":
         print("Got SAML %s, opening browser..." % sam, file=stderr)
     slv = SAMLLoginView(uri, html, verbose=args.verbose, cookies=args.cookies, verify=args.verify)
     Gtk.main()
         print("Got SAML %s, opening browser..." % sam, file=stderr)
     slv = SAMLLoginView(uri, html, verbose=args.verbose, cookies=args.cookies, verify=args.verify)
     Gtk.main()

+    if slv.closed:
+        print("Login window closed by user.", file=stderr)
+        p.exit(1)

     if not slv.success:
     if not slv.success:

-        p.error('''Login window closed without producing SAML cookie''')
+        p.error('''Login window closed without producing SAML cookies.''')

 
     # extract response and convert to OpenConnect command-line
     un = slv.saml_result.get('saml-username')
 
     # extract response and convert to OpenConnect command-line
     un = slv.saml_result.get('saml-username')

-    for cn in ('prelogin-cookie', 'portal-userauthcookie'):
+    server = slv.saml_result.get('server', args.server)
+
+    for cn, ifh in (('prelogin-cookie','gateway'), ('portal-userauthcookie','portal')):

         cv = slv.saml_result.get(cn)
         if cv:
             break
     else:
         cv = slv.saml_result.get(cn)
         if cv:
             break
     else:

-        cn = None
+        cn = ifh = None
+        p.error("Didn't get an expected cookie. Something went wrong.")

 
     if args.verbose:
 
     if args.verbose:

-        print('''\n\nSAML response converted to OpenConnect command line invocation:\n''', file=stderr)
-        print('''    echo {!r} |\n        openconnect --protocol=gp --user={!r} --usergroup={}:{} --passwd-on-stdin {}\n'''.format(
-            cv, un, ('portal' if args.portal else 'gateway'), cn, args.server), file=stderr)
+        # Warn about ambiguities
+        if server != args.server and not args.uri:
+            print('''IMPORTANT: During the SAML auth, you were redirected from {} to {}. This probably'''
+                  '''means you should specify {} as the server for final connection, but we're not 100%'''
+                  '''sure about this. You should probably try both.\n'''.format(args.server, server), file=stderr)
+        if ifh != args.interface and not args.uri:
+            print('''IMPORTANT: We started with SAML auth to the {} interface, but received a cookie'''
+                  '''that's associated with the {} interface. You should probably try both.\n'''.format(args.interface, ifh),
+                  file=stderr)
+
+        print('''\nSAML response converted to OpenConnect command line invocation:\n''', file=stderr)
+        print('''    echo {} |\n        openconnect --protocol=gp --user={} --usergroup={}:{} --passwd-on-stdin {}'''.format(
+            quote(cv), quote(un), quote(ifh), quote(cn), quote(server)), file=stderr)
+
+        print('''\nSAML response converted to test-globalprotect-login.py invocation:\n''', file=stderr)
+        print('''    test-globalprotect-login.py --user={} -p '' \\\n         https://{}{} {}={}\n'''.format(
+            quote(un), quote(server), quote(if2auth[ifh]), quote(cn), quote(cv)), file=stderr)

 
 

-    print("HOST={!r}\nUSER={!r}\nCOOKIE={!r}".format('https://%s/%s:%s' % (args.server, ('portal' if args.portal else 'gateway'), cn), un, cv))
+    varvals = {
+        'HOST': quote('https://%s/%s:%s' % (server, if2auth[ifh], cn)),
+        'USER': quote(un), 'COOKIE': quote(cv),
+    }
+    print('\n'.join('%s=%s' % pair for pair in varvals.items()))