Changeset 62f3dd9 for fedd/federation

Timestamp:

Dec 10, 2010 9:00:16 AM (13 years ago)

Author:

Ted Faber <faber@…>

Branches:

axis_example, compt_changes, info-ops, master

Children:

913dc7a

Parents:

fbee30a

git-author:

Ted Faber <faber@…> (12/09/10 11:41:45)

git-committer:

Ted Faber <faber@…> (12/10/10 09:00:16)

Message:

allow command line progams to expand tildes. Added a class derived from OptionParser? to make that easily available.

Location:

fedd/federation

Files:

• client_lib.py (modified) (4 diffs)
• util.py (modified) (2 diffs)

fedd/federation/client_lib.py

@@ -10 +10 @@
 
 from fedid import fedid
-from util import fedd_ssl_context
+from util import fedd_ssl_context, file_expanding_opts
 from remote_service import service_caller
 from service_error import service_error
@@ -17 +17 @@
 
 
-class client_opts(OptionParser):
+class client_opts(file_expanding_opts):
     """
     Standatd set of options that all clients talking to fedd can probably use.
     Client code usually specializes this.
     """
+
     def __init__(self):
-        OptionParser.__init__(self, usage="%prog [opts] (--help for details)",
+        file_expanding_opts.__init__(self,
+                usage="%prog [opts] (--help for details)",
                 version="0.1")
 
-        self.add_option("--cert", action="store", dest="cert",
+        self.add_option("--cert", action="callback", dest="cert", 
+                callback=self.expand_file,
                 type="string", help="my certificate file")
-        self.add_option("--abac", action="store", dest="abac_dir",
-                type="string", help="Directory with abac certs")
+        self.add_option("--abac", action="callback", dest="abac_dir",
+                callback=self.expand_file,
+                type="string", default=os.path.expanduser('~/.abac'), 
+                help="Directory with abac certs")
+        self.add_option('--no_abac', action='store_const', const=None, 
+                dest='abac_dir', help='Do not use abac authorization')
         self.add_option( "--debug", action="count", dest="debug", 
                 default=0, help="Set debug.  Repeat for more information")
@@ -35 +42 @@
                 dest="serialize_only", default=False,
                 help="Print the SOAP request that would be sent and exit")
-        self.add_option("--trusted", action="store", dest="trusted",
+        self.add_option("--trusted", action="callback", dest="trusted",
+                callback=self.expand_file,
                 type="string", help="Trusted certificates (required)")
         self.add_option("--url", action="store", dest="url",
@@ -98 +106 @@
     '''
     rv = [ ]
-    if dir:
+    if dir and os.path.isdir(dir):
         for fn in ["%s/%s" % (dir, p) for p in os.listdir(dir) \
                 if os.path.isfile("%s/%s" % (dir,p))]:

fedd/federation/util.py

@@ -8 +8 @@
 
 import httplib
+
+from optparse import OptionParser
 
 from socket import sslerror
@@ -95 +97 @@
             self.set_allow_unknown_ca(True)
             self.set_verify(SSL.verify_peer, 10, callback=callb)
+
+class file_expanding_opts(OptionParser):
+    def expand_file(self, option, opt_str, v, p):
+        """
+        Store the given value to the given destination after expanding home
+        directories.
+        """
+        setattr(p.values, option.dest, os.path.expanduser(v))
+
+    def __init__(self, usage=None, version=None):
+        OptionParser.__init__(self)
+
 
 def read_simple_accessdb(fn, auth, mask=[]):