• last updated 4 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Provide default help texts for register form

Prevent error when user is removed from the Registered Users group, but not from users altogether

Fix typo in previous commit

Don't steal anybody's address nor email, especially if people was banned. Fail and give a message instead.

simplify idiom

Provide more infrastructure support for host-node-maps for more flexible domain handling

- add optional -cookie_domain parameter to the following functions

ad_user_login

ad_user_logout

sec_generate_session_id_cookie

auth::issue_login (wrapper for ad_user_login)

if not specified, the functions are full backward compatible

- add optional parameter -host_node_id to auth::authenticate

which refers to the node_id in the host-node-map

- pass host_node_id from from/to register pages

TODO:

1) probably, sec_generate_session_id_cookie picks up the wrong

session_id via [ad_conn session_id]

2) check interaction with CookieDomain (probably, we need

must not pass host_node_id wheren CookieDomain is non-empty)

  1. … 6 more files in changeset.
Increase size of URL, remove default, since it does not match well with required/non-required

Fix spelling error

Fix spelling errors

  1. … 24 more files in changeset.
White space change: remove tabs and trailing spaces

    • -300
    • +298
    ./authentication-procs.tcl
Improve comment, remove useless parenthesis

Remove (harmless) inconsistency in local authority service contract (many thanks to Günther Ernst)

Use "in" idiom instead of lsearch

- replace deprecated "cc_lookup_email_user ..." by "party::get_by_email -email ..."

  1. … 1 more file in changeset.
- reduce the dependencies on acs_object_party_privilege_map for PostgreSQL

  1. … 1 more file in changeset.
- fix test case

- prefer empty sql statements over various forms of *SQL*

- no need to use backslash for continuation between curly brackets

- improve readability of code

- modernize tcl

  1. … 5 more files in changeset.
- add editor hints to keep spaces/tabs in the furture more consistent

  1. … 745 more files in changeset.
- remove globals for errorInfo and use namespace qualifiers instead

  1. … 10 more files in changeset.

ad_proc reform:

- use nsf::proc for defining ad_procs, when nsf::proc is available

- nsf::proc is part of the environment of XOTcl2 and provides

an efficient, C-implemented argument parser that can replace

the scripted *__arg_parser of OpenACS.

- The benefits are performance and reduced memory consumption.

* Simple functions (where the calling overhead was a large

part of the execution times can become several times faster

(e.g. db_name is 3x faster, cookie handling is 2x faster),

for functions with much db-activities the percentage will

be less.

* The memory consumption (RSS) of a site like OpenACS.org

is reduced by about 15%.

NEW (with nsf::proc)

::ad_get_cookie x: 10.93 microseconds per iteration

::ad_get_cookie ad_session_id: 21.95 microseconds per iteration

::dt_systime: 24.89 microseconds per iteration

::db_name: 8.70 microseconds per iteration

::db_string dbqd.null.g "select version() from dual": 316.66 microseconds per iteration

::ad_convert_to_html "hello world": 102.29 microseconds per iteration

OLD (without nsf::proc)

::ad_get_cookie x: 19.42 microseconds per iteration

::ad_get_cookie ad_session_id: 30.10 microseconds per iteration

::dt_systime: 27.55 microseconds per iteration

::db_name: 32.28 microseconds per iteration

::db_string dbqd.null.g "select version() from dual": 366.69 microseconds per iteration

::ad_convert_to_html "hello world": 160.51 microseconds per iteration

======================================================================

proc mtime cmd {set c 5000

set t [time [list time $cmd $c]];

regexp {^(-?[0-9]+) +} $t _ mS1;

set ms [expr {$mS1*1.0/$c}];

return "[format %6.2f $ms] microseconds per iteration"

}

set tests {

{::ad_get_cookie x} {::ad_get_cookie ad_session_id} {::dt_systime}

{::db_name} {::db_string dbqd.null.g "select version() from dual"}

{::ad_convert_to_html "hello world"}

}

foreach t $tests {append ::_ $t ": " [mtime $t] \n}

set ::_

======================================================================

    • -18
    • +10
    ./test/acs-authentication-procs.tcl
  1. … 8 more files in changeset.
- fix for bug #3251 from Andrew Helsley

  1. … 5 more files in changeset.
Merging back to HEAD all changes that happened in branch oacs-5-8 between tags: vg-merge-oacs-5-8-from-20141027 and vg-merge-oacs-5-8-from-20150427

  1. … 520 more files in changeset.
Merging back to HEAD branch oacs-5-8 (using tag vg-merge-oacs-5-8-from-20141027).

    • -0
    • +28
    ./authentication-procs-postgresql.xql
    • -19
    • +23
    ./test/acs-authentication-procs.tcl
  1. … 2538 more files in changeset.
- simplify code

  1. … 2 more files in changeset.
- use util::http::get instead of deprecated proc

- use $::acs::rootdir rather than [acs_root_dir] or [get_server_root]

  1. … 12 more files in changeset.
- reduce number of remaining "string equal" operations

  1. … 2 more files in changeset.
- flatten nested "lindex" structures

- use numeric comparison operator for truth value comparisons

  1. … 1 more file in changeset.