Age | Commit message (Collapse) | Author | |
---|---|---|---|
2013-03-09 | Preserve (instead of collapse) arguments in su_monkeysphere_user().autoquote | Jonas Smedegaard | |
It is a healthy coding practice to keep each argument separate when executing system calls, i.e. quote each variable separately instead of relying on whitespace to indicate argument separation. Quoting shell-inside-shell is tricky to do right, but not impossible: Bourne-derived shells treat single-quoting literally, which means that shell command arguments (i.e. an array of strings) should be safe to serialize (dual-quote) using these simple rules: b) each single-quote inside each string is escaped as '\'' a) each string is surrounded by single-quotes This patch applies above single-quote serialization for su_monkeysphere_user(), and adapts ma/update_users $STRICT_MODES on line 82 to match. If serialization routine should turn out to be broken, it will therefore only affect $STRICT_MODES in ma/update_users badly. All other variables passed through su_monkeysphere_user() should not possibly be treated any worse than previous non-quoting. | |||
2013-03-09 | Simplify arguments passed to su_monkeysphere_user() and gpg_sphere | Jonas Smedegaard | |
It is a healthy coding practice to keep each argument separate when executing system calls, i.e. quote each variable separately instead of relying on whitespace to indicate argument separation. This patch improves argument passing like this: a) Each argument is passed individually (not all as a single string) b) Arguments containing no variables are not quoted c) Dynamic arguments are double-quoted ( "$@" ) Due to su_monkeysphere_user() expanding arguments using "$*" (not "$@") arguments are mostly¹ coollapsed into single strings, and this change is therefore only cosmetic. It does improve clarity, however. Also, it eases switching to safer quoted arguments in the future. ¹As sole excepion ma/update_users line 82 has $STRICT_MODES dual-quoted. | |||
2013-03-09 | Pass only single commands through su wrapper | Jonas Smedegaard | |
It is a healthy coding practice to keep each argument separate when executing system calls, i.e. quote each variable separately instead of relying on whitespace to indicate argument separation. Quoting shell-inside-shell is tricky to do right, and gets trickier when more than a single command is wrapped together. This patch simplifies convoluted shell calls to contain only one command each. | |||
2013-02-07 | added keyid-format 0xlong to monkeysphere-authentication default gpg.conf ↵ | Daniel Kahn Gillmor | |
files (thanks, Jonas!) | |||
2013-02-06 | incorporate example combined ProxyCommand from sanoj_ | Daniel Kahn Gillmor | |
2013-01-18 | enable openpgp2pem as well from keytrans | Daniel Kahn Gillmor | |
2012-09-11 | fetch all keys instead of the first 5 (and work better with gpg 2.0.19, ↵ | Daniel Kahn Gillmor | |
which apparently does not retrieve keys from gpg --search if the --batch argument is also present) | |||
2012-03-20 | add --batch to gpg invocations, since gpg2 requires it to use --passphrase-* ↵ | Daniel Kahn Gillmor | |
arguments | |||
2012-03-16 | use date somewhat more portably | Daniel Kahn Gillmor | |
2012-02-24 | fix url in man page, fixes #635648 | Micah Anderson | |
2011-01-02 | genericized x509 certificate generation -- now works for any service, not ↵ | Daniel Kahn Gillmor | |
just https (invoke it like "make-x509-certreqs imap" for imap:// keys, etc) | |||
2010-12-21 | avoid problems with filenames containing regexp special characters. | Jamie McClelland | |
2010-12-21 | On dkg's suggestion, using hash index as more elegant way to ensure | Jamie McClelland | |
we don't repeat users. | |||
2010-12-21 | Tracking users while generating watch list is more reliable | Jamie McClelland | |
way to ensure m-a u is executed on the right users. | |||
2010-12-20 | adding some comments/concerns about the current monkeysphere-monitor-keys ↵ | Daniel Kahn Gillmor | |
implementation | |||
2010-12-20 | added example script to auto-generate X.509 certificate requests with the ↵ | Daniel Kahn Gillmor | |
PGPExtension embedded in them from https monkeysphere-host keys. | |||
2010-12-20 | update debug to use printf and always emit newlines; use warnings; | Daniel Kahn Gillmor | |
2010-12-20 | whitespace, simple style cleanup | Daniel Kahn Gillmor | |
2010-12-20 | Merge remote branch 'jamie/master' | Daniel Kahn Gillmor | |
2010-12-20 | keytrans: avoid confusing user IDs across different keys (closes MS # 2682) | Daniel Kahn Gillmor | |
2010-12-20 | Comments at top now contains more concrete explanation of how | Jamie McClelland | |
the script works. Location of key files to monitor is more configurable by the sys admin. All changed files treated the same for simplicity. Added debug mode. | |||
2010-12-20 | more precise description of file::ChangeNotify's behavior | Jamie McClelland | |
with regard to different operating systems. And, monitoring /etc/passwd is not recommended, so removed from description. | |||
2010-12-19 | first attempt at closing #499 (changes to user authorized_keys files not | Jamie McClelland | |
immediately incorporated) | |||
2010-11-13 | update changelog | Jameson Rollins | |
2010-11-13 | If for whatever reason the primary UID comes up empty, give the injected ↵ | Daniel Kahn Gillmor | |
subkey a reasonable name instead of the empty string | |||
2010-11-13 | universalize and consolidate on --fixed-list-mode | Daniel Kahn Gillmor | |
2010-11-13 | avoid using the running MSVA from the user during the tests | Daniel Kahn Gillmor | |
2010-10-30 | update upstream changelog | Jameson Rollins | |
2010-10-30 | clean up ssh_proxycommand function (no functional change) | Jameson Rollins | |
2010-10-30 | break out proxy command validation code into it's own function (no ↵ | Jameson Rollins | |
functional change) | |||
2010-10-30 | improve debug output | Jameson Rollins | |
2010-10-30 | fix variable declarations in update_known_hosts | Jameson Rollins | |
2010-10-30 | catch return for grep in remove_line, for case where grep -v returns nothing ↵ | Jameson Rollins | |
(e.g. only line in file is removed) | |||
2010-10-29 | fix ssh_proxycommand marginal ui | Jameson Rollins | |
After the last big code cleanup, the bookkeeping of numbers of processed/valid keys was removed. This was done because most things don't use that info, and it was confusing return codes of top-level functions. The one thing that did use that, though, was the ssh_proxycommand. We fix this by using a global variable to keep track of the number of processed and valid keys. The proxy command can now easily determine when it should output it's marginal ui block. | |||
2010-10-29 | fix variable declaration (leftover from break out of touch_key_file_or_fail) | Jameson Rollins | |
2010-10-29 | consolidate and simplify printing of key lines in process_keys_for_file | Jameson Rollins | |
also move hashing of known_hosts lines into ssh2known_hosts function | |||
2010-10-27 | remove reference to USE_VALIDATION_AGENT (the usage was incorrect as well) | Jameson Rollins | |
2010-10-27 | fix label in upstream changelog | Jameson Rollins | |
2010-10-26 | finalize changelog for 0.34 release | Jameson Rollins | |
2010-10-26 | add note about CHECK_KEYSERVER var in msph-auth man page | Jameson Rollins | |
2010-10-25 | ensure that we only remove fully-matching lines once we have found them | Daniel Kahn Gillmor | |
2010-10-24 | back to using grep fixed-string matching when removing key lines | Jameson Rollins | |
This method uses grep -F to find the full line to match, and then second call to grep -v -F to actually remove the line. For known_hosts, we use two piped grep -F calls. No rexexp are used, and only one extra call to grep is required for known_hosts line removal. There is still an issue here about sub-string matches, but there is at least no regression over early versions. | |||
2010-10-24 | don't fail if authorized_keys file not present | Jameson Rollins | |
we create a new function here, touch_key_file_or_fail, which will touch a new key file if there isn't one already present. This is now used in the update_authorized_keys and update_known_hosts functions when looking for authorized_keys and known_hosts respectively. Closes Debian 600644 | |||
2010-10-22 | tweak log levels and messages in ma/update_users | Jameson Rollins | |
2010-10-19 | fix remove_line function to not use fixed string checking, and to mv -f the ↵ | Jameson Rollins | |
tmp file into place | |||
2010-10-18 | fix remove_monkeysphere_lines function to just read from stdin and write to ↵ | Jameson Rollins | |
stdout | |||
2010-10-18 | fix update_known_hosts to create proper initial temp file | Jameson Rollins | |
2010-10-18 | fix back to integer indexing in process_authorized_user_ids | Jameson Rollins | |
2010-10-18 | cleanup update_known_hosts | Jameson Rollins | |
* don't update if unchanged * proper trap setting * cleanup comments | |||
2010-10-18 | fix up update_authorized_keys | Jameson Rollins | |
* better trap handling * don't update file if unchanged * clean up comments |