summaryrefslogtreecommitdiffstats
path: root/usr.sbin/bsdconfig/usermgmt
Commit message (Collapse)AuthorAgeFilesLines
* MFC revisions 260894,260899,262895-262902,262904,262908-262910,262982,dteske2014-04-0115-1495/+2961
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 262984,263133-263137,263139,263141,263144-263150, and (partially) 263249 (31 revisions; summarized below)... r260894: Optimize f_expand_number(), improving performance r260899: s/__number/__num/ in f_expand_number() r262895: Allow dispatched reswords to carry arguments r262896: Add missing local declaration r262897: Fix a typo in a comment r262898: Fix incorrect return status if var_to_set and var_to_get are same r262899: Make f_show_err non-fatal r262900: Centralize function name; Update a comment while here r262901: s/__num/__number/ in f_expand_number() r262902: Comment to go with NL global introduced by previous commit r262904: Rewrite groupmgmt -- hooking it into the scripting system r262908: Change dispatch words from group* to *Group for backward compat r262909: Fix copy/paste error in a comment r262910: Take a group name on the command-line if available r262982: Whitespace r262984: Remove vestigial global, no longer used since r262904 r263133: Remove indexfile from debug statement (already logged) r263134: Add debug statement just before attempting to exec a module r263135: Comments r263136: Update copyright r263137: Fix future namespace issues for functions taking $var_to_set r263139: Remove useless NULL string in compound strings r263141: Pointy hat! Fix a broken f_isinteger() r263144: Fix a code-typo that prevented auto-sizing of a dialog r263145: Fix comments and whitespace r263146: Reduce the sleep cycle when using dialog(1) [infobox] to 1-second r263147: Fix a bug preventing errors from pw(8) from appearing r263148: For non-interactive scripts, forgot to check argument r263149: Add protection against input containing single-quotes r263150: Rewrite usermgmt r263249: (partial) Add more obsolete files
* MFC r259054:dteske2014-01-1511-110/+74
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Performance and debugging enhancements: + Remove UNAME_P=$(...) from startup/misc -- already supplied by common.subr + Use f_getvar instead of $(eval echo \$$var) -- f_getvar is sub-shell free + Add `-e' and `-k var' options to f_eval_catch -- increasing use-cases + Use f_eval_catch to display errors on failure -- reducing duplicated code + Use f_eval_catch when we need output from a command -- improving debugging + Optimize f_isinter of strings.subr for performance -- now sub-shell free + Improve error checking on pidfiles -- using f_eval_catch and f_isinteger + Use $var_to_set arg of f_ifconfig_{inet,netmask} -- eliminate sub-shells + Use f_sprintf instead of $(printf ...) -- consolidate sub-shells + Use $var_to_set arg of f_route_get_default -- eliminate sub-shells + Add f_count to replace $(set -- ...;echo $#) -- eliminate sub-shells + Add f_count_ifs to replace $(IFS=x;set -- ...;echo $#) -- no sub-shells + Replace var="$var${var:+ }..." in loops with var="$var ..." with a follow- up var="${var# }" to trim leading whitespace -- optimize loops + Use $var_to_set arg of f_resolv_conf_nameservers -- eliminate sub-shells + Comments for the f_eval_catch function + Remove a duplicate `local ... desc ...' in f_device_get_all of device.subr + Use $var_to_set arg of f_device_capacity -- eliminate sub-shells + Whitespace fixes in f_dialog_init of dialog.subr + Optimize f_inet_atoi of media/tcpip.subr for performance -- sub-shell free + In several cases, send stderr to /dev/null -- clean up runtime execution + Change f_err of common.subr to go to program stderr not terminal stderr, allowing redirection of output from functions that use f_err + Disable debugging when using f_getvar to get variable argument to f_startup_rcconf_map_expand of startup/rcconf.subr + Use f_replace_all instead of $(echo ... | tr | sed) -- performance + Add a $var_to_set option to f_index_{file,menusel_{command,keyword}} of common.subr -- centralize sub-shells
* MFC r258355,258360,258400-258401,258406-258407,258418,258430,258439,258589-dteske2014-01-151-5/+5
| | | | | | | | | | | | | | | | | 258590,258592,258727-258728,258785, with the following descriptions: Revisions 258400-258401,258406-258407,258590,258727-258728,258785: + Add a new `includes' module (see: bsdconfig includes -h) for exploring the bsdconfig(8) API Remaining revisions: 258592: Sort all the messages.subr files 258355: Fix typo in a comment in networking/device.subr 258360: Whitespace, style, and other changes in networking/device.subr 258418: Quote interface names for good measure in networking/device.subr 258430: Remove an unused line in networking/device.subr 258439: Minor bugfix to f_die() function in common.subr 258589: Minor bugfix to syscons_ttys module
* Introduce centralized [X]dialog(1) exit codes to `dialog.subr' and make-usedteske2013-10-099-70/+67
| | | | | | | | | | throughout the bsdconfig(8) code. While we're here, add an explicit argument to lvalue-seeking invocations of "return" that previously had no argument. Also, consolidate a single instance of double-newline and remove some comments that are no longer required (given increased readability with new exit codes). Approved by: re (glebius)
* Fix typos in the BSD License.dteske2013-07-071-2/+2
|
* Be consistent with other usr.sbin programs w/respect to the copyrightdteske2013-07-071-1/+1
| | | | wording ("All rights reserved."); I had the casing wrong on many instances.
* Use f_show_msg() instead of f_dialog_msgbox() where appropriate. The maindteske2013-07-054-18/+16
| | | | | | | | | | | | difference between these two functions: Usage: f_show_msg() $format_string [ $format_args ... ] Usage: f_dialog_msgbox() $text [ $hline ] The former lends itself well to displaying the $msg_* i18n text, prompts, etc. While the latter is better for text you do not control (error strings captured as a response from external commands) -- or if you have to control the hline.
* More whitespace.dteske2013-06-2410-17/+17
|
* When the fall-back of a case-statement is the last thing executed in adteske2013-06-201-7/+6
| | | | | | | | while-loop _and_ all prior matches in the same case-statement either break or continue, we can safely break the fall-back out of the case-statement. This should improve readability and allow for longer-lines by reducing the level of indentation by-one for the fall-back case. (a continuation of SVN r252019)
* When the fall-back of a case-statement is the last thing executed in adteske2013-06-203-21/+18
| | | | | | | while-loop _and_ all prior matches in the same case-statement either break or continue, we can safely break the fall-back out of the case-statement. This should improve readability and allow for longer-lines by reducing the level of indentation by-one for the fall-back case.
* Whitespace.dteske2013-06-191-20/+10
|
* Whitespace and comments.dteske2013-06-191-1/+0
|
* Remove extra whitespace lines.dteske2013-06-194-8/+0
|
* Perform some code consolidation and add some additional errordteske2013-06-182-28/+17
| | | | checking/reporting. (similar to SVN revisions 251919 and 251928)
* Improve the INDEX format. Whitespace improvements, format improvements, typodteske2013-06-181-17/+21
| | | | and grammatical fixes.
* Add a prompt to the "Add User" and "Add Group" sub-menu items (under thedteske2013-06-083-34/+64
| | | | | | | | | | "Login Management" module): Use default values for all account details? If you select "Yes" (the default is "No"), you'll jump past all the prompts and jump straight to the review screen with all-default values. Makes adding a lot of users/groups faster/easier if you don't need to customize more than one or two different values from their defaults.
* Curb another regression (this one introduced by r251545, itself trying todteske2013-06-082-6/+14
| | | | | | | | | | | | | | fix the regression introduced by r251544; which was trying to make things consistent w/respect to ESC versus YES versus NO in the password disable prompt in "Login Management". (need stronger coffee; very sorry for the churn) With this revision, the "YES", "NO", and implied ESC options all work as- expected. Choosing "YES" allows you to proceed and the password will be disabled. Choosing "NO" will bring back around to enter a password for the account. Pressing ESC will drop you out of either user or group input and back to the usermgmt screen.
* Fix a regression introduced by r251544; if user chooses "NO" when beingdteske2013-06-082-0/+2
| | | | | asked if they would like to disable password authentication for the account, loop back to prompt them for the password again.
* Check for ESC as a separate choice from "NO" when asking if the operatordteske2013-06-082-8/+8
| | | | | | | | | | | | | would like to disable password management for an account while adding either a user or group. When being prompted to answer questions while adding a group or user, two things are trow: 1. You can hit ENTER to blast through all the questions and in the end, the group or user is created with sensible defaults for all values. 2. You can press ESC during any prompt to cancel the operation as a whole. This fix is shoring up an inconsistency in the latter (#2).
* Track defaultitem in the password and account expiry menus; if the userdteske2013-06-081-2/+6
| | | | | presses ESC or chooses Cancel/No in any of the sub-menus, we want to return to the expiry menu with that item selected.
* Fix a regression in the "Login Management" module introduced by r251242 indteske2013-06-081-4/+8
| | | | | | which choosing to cancel the manual input of expiration time (in seconds since the UNIX epoch) for either account expiration or password expiration would see the original value lost.
* Change the name of a couple of functions in `usermgmt/user_input.subr' todteske2013-06-022-8/+8
| | | | | | | | | | | | | | make their purpose more clear by their names: f_dialog_input_change() and f_dialog_input_expire() Are now (respectively): f_dialog_input_expire_password() and f_dialog_input_expire_account() Upon revisit to this portion of code, the former names were too confusing.
* Improvements to whitespace and comments.dteske2013-06-022-17/+17
|
* Standardize the way functions build their arguments leading up to a dialogdteske2013-06-023-114/+110
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | invocation. Specifically, "top-load" your arguments and in the order in- which they will be displayed. For example, many [if not all] widgets display information in the following order, top-to-bottom (visually): + backtitle (displayed behind the widget at top-left) + title (at the top of the `window') + prompt text (just below the title and above whatever widget you choose) + Depending on widget, _one_ of the following: - menu list - radio list - check list - text input box with initial text - [Xdialog(1)] 2x or 3x text input boxes - [dialog(1)] a multi-part form - progress bar - etc. (many more widget choices) + buttons (right below the selected widget) + [dialog(1)] the hline (displayed at bottom of `window') NOTE: Xdialog(1) accepts and silently ignores --hline When building local arguments for your dialog invocation, if the value can't be cleanly loaded into a local, add "# Calculated below" to the end of the local declaration while retaining the block order of argument declarations. Move other local declarations that are not associated with this top-loading the dialog arguments to right-above where they are first-used. Also, standardize on the names of the arguments. For example, always use $prompt (instead of sometimes $msg and sometimes $prompt); use $menu_list or $shell_list or $radio_list for those respective widgets; ad nauseum. While we're doing this, flush-out full arguments for many invocations that were passing NULL strings (making it unapparent if you were staring at this one invocation what argument that NULL string was supposed to represent). Last, while we're in startup/rcconf let's remove the unnecessary use of a GLOBAL (RCCONF_MENU_LIST) for the menu_list.
* Improve the dialog(1) API in dialog.subr by adding f_dialog_default_store()dteske2013-06-023-32/+38
| | | | | | | | | | and f_dialog_default_fetch(). Operating similar to functions introduced by SVN r251236 and r251242, these functions operate as a pair for helping track the default-item data (for the --menu, --checklist, and --radiolist widgets). This replaces the direct usage of a global to store the data with an abstract method for readability and to centralize the code.
* Similar to r251236, improve the portion of dialog(1) API in dialog.subrdteske2013-06-022-73/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | responsible for retrieving stored input (for the --inputbox and --password widgets). When we (Ron McDowell and I) developed the first version of bsdconfig, it used temporary files to store responses from dialog(1). That hasn't been true for a very long time, so the need to always execute some clean-up function is long-deprecated. The function that used to perform these clean- up routines for these widgets was f_dialog_inputstr(). We really don't need f_dialog_inputstr() for its originally designed purpose as all dialog invocations no longer require temporary files. Just as in r251236, redesign f_dialog_inputstr() in the following four ways: 1. Rename f_dialog_inputstr() to f_dialog_inputstr_fetch() 2. Introduce the new first-argument of $var_to_set to reduce forking 3. Create a corresponding f_dialog_inputstr_store() to abstract storage 4. Offload the sanitization to a new function, f_dialog_line_sanitize() It should be noted that f_dialog_line_sanitize() -- unlike its cousin from SVN r251236, f_dialog_data_sanitize() -- trims leading/trailing whitespace from the user's input. This helps prevent errors and common mistakes caused by the fact that the new cdialog implementation allows the right-arrow cursor key to go beyond the last byte of realtime input (adding whitespace at the end of the typed value). While we're centralizing the sanitization, let's rewrite f_dialog_input() while we're here to likewise reduce forking. The f_dialog_input() function now expects the first argument of $var_to_set instead of producing results on standard-out. These changes greatly improve readability and also improve performance.
* Improve portion of the dialog(1) API in dialog.subr responsible fordteske2013-06-019-86/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | retrieving stored data (for the --menu, --calendar, --timebox, --checklist, and --radiolist widgets). When we (Ron McDowell and I) developed the first version of bsdconfig, it used temporary files to store responses from dialog(1). That hasn't been true for some very long time, so the need to always store the return status of dialog(1) and then call some function to clean-up is long-deprecated. The function that used to do the clean-up was f_dialog_menutag(). We really don't need f_dialog_menutag() for its originally designed purpose, as all dialog invocations (even when in a sub-shell) do not use temporary files anymore. However, we do need to keep f_dialog_menutag() around because it still fills the need of being able to abstract the procedure for fetching stored data provided by functions that display the aforementioned widgets. In re-designing f_dialog_menutag(), four important changes are made: 1. Rename f_dialog_menutag() to f_dialog_menutag_fetch() 2. Introduce the new first-argument of $var_to_set to reduce number of forks 3. Create a corresponding f_dialog_menutag_store() to abstract the storage 4. Offload the sanitization to a new function, f_dialog_data_sanitize() NOTE: That last one is important. Not all functions need to store their data for later fetching, meanwhile every invocation of dialog should be sanitized (as we learned early-on in the i18n-effort -- underlying libraries will spit warnings to stderr for bad values of $LANG and since dialog outputs its responses to stderr, we need to sanitize every response of these warnings). These changes greatly improve readbaility and also improve performance by reducing unnecessary forking.
* Improve portion of the dialog(1) API in dialog.subr responsible fordteske2013-05-315-165/+195
| | | | | | | | | | | | | | | | | | | | calculating widget sizes. Instead of forking a sub-shell to calculate the optimum size for a widget, use a byRef style call-out to set variables in the parent namespace. For example, instead of: size=$( f_dialog_buttonbox_size title btitle msg ) $DIALOG --title title --backtitle btitle --msgbox msg $size The new API replaces the above with the following: f_dialog_buttonbox_size height width title btitle msg $DIALOG --title title --backtitle btitle --msgbox msg $height $width This reduces the number of forks, improves performance, and makes the code more readable by revealing the argument-order for widget sizing. It also makes performing minor adjustments to the calculated values easier as you no longer have to split-out the response (which required knowledge of ordering so was counter-intuitive).
* Centralize standard getopts arguments, both for convenience and to correctdteske2013-05-147-7/+7
| | | | | | | | | a bug in which certain combinations of arguments produced unexpected results such as `-dX' (now properly produces debugging and X11), `-XS' (now properly produces X11 in secure mode), `-df-' (enables debugging when reading a script from standard-input, etc. Multi-word variations such as `-d -X', `-X -S', `-d -f-', `-d -f -', etc. also work as expected. Also tested were variations in argument order, which are now working as expected.
* UI improvements. First, implement --default-item whenever and whereverdteske2013-04-2210-42/+68
| | | | | | | | possible to save keystrokes. Second, overhaul startup/rcdelete for much improved performance. Last, but not least, kill-off useage of --clear and implement --keep-tite in harmony to minimize jarring transitions. Also, fix local variable names where necessary while we're here with other minor comment-enhancements/typo-corrections.
* Add new flags `-d' (sets debug=1) and `-D file' (sets debugFile) anddteske2013-04-228-16/+20
| | | | | | improve debugging initialization. Also fixup USAGE statements while we're here. Also, change initialization of main program to _not_ change working directory, allowing the debugFile to be relative without confusion.
* Don't use f_show_msg() unless printf(1) syntax is required (this reduces thedteske2013-01-144-32/+34
| | | | number of unnecessary forks).
* Add missing dialog(1) arguments to internationalize "Help"-button label.dteske2013-01-141-0/+1
|
* Add more debugging to help with diagnosis of program-flow when needed.dteske2012-12-2511-0/+19
|
* Remove unnecessary duplicate initialization of the dialog(1) API (automaticallydteske2012-12-259-9/+0
| | | | bootstrapped on-include unless DIALOG_SELF_INITIALIZE is set to NO before-hand)
* Remove erroneous use of "local" (not in a function).dteske2012-12-211-1/+0
|
* Improve the debugging abilities and clean up debug messages. In most cases,dteske2012-12-219-55/+60
| | | | | all one has to do is set the environment variable DEBUGGING to get the debug messages to appear on the console.
* Add much-needed dialog(1) --inputbox function for simplifying the process ofdteske2012-12-212-335/+48
| | | | requesting input from the user.
* Replicate a feature from sysinstall documented indteske2012-11-169-9/+9
| | | | | | | | | | | | stable/9/usr.sbin/sysinstall/help/shortcuts.hlp (reproduced below): If /usr/sbin/sysinstall is linked to another filename, say `/usr/local/bin/configPackages', then the basename will be used as an implicit command name. To get a list of modules you can link to, see `bsdconfig -h' output. Approved by: adrian (co-mentor) (implicit)
* Resurrect and integrate stable/9/usr.sbin/sysinstall/help/usermgmt.hlpdteske2012-10-263-2/+96
| | | | Approved by: adrian (co-mentor) (implicit)
* Reverse SVN r240770 -- jilles@ made a suggestion that allowed us to redesigndteske2012-09-211-6/+6
| | | | | | | our embedded rescue environment to support /dev/null making r240770 obsolete. Reviewed by: jilles, adrian (co-mentor) Approved by: jilles, adrian (co-mentor)
* Replace redirections to /dev/null with "close file-descriptor" syntax (>&-).dteske2012-09-211-6/+6
| | | | | Reviewed by: adrian (co-mentor) Approved by: adrian (co-mentor)
* Change all invocations of dialog(1) to no-longer require temporary files.dteske2012-09-205-64/+162
| | | | | | | This allows bsdconfig to -- like bsdinstall -- operate from read-only media. Reviewed by: adrian (co-mentor) Approved by: adrian (co-mentor)
* Move major includes into /usr/share/bsdconfig for easy external access.dteske2012-09-1814-63/+74
| | | | | Reviewed by: adrian (co-mentor) Approved by: adrian (co-mentor)
* Import bsdconfig(8) as a replacement for the post-install abilities ofdteske2012-07-1416-0/+3309
deprecated sysinstall(8). NOTE: WITH_BSDCONFIG is currently required. Submitted by: Devin Teske (dteske), Ron McDowell <rcm@fuzzwad.org> Reviewed by: Ron McDowell <rcm@fuzzwad.org> Approved by: Ed Maste (emaste)
OpenPOWER on IntegriCloud