• last updated 11 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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.
- brace expressions to allow for tcl compilation

- use "in" or not in (ni) operator instead of lsearch (for readability and performance)

  1. … 10 more files in changeset.
- use "in" or not in (ni) operator instead of lsearch (for readability and performance)

  1. … 2 more files in changeset.