• last updated 6 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Bring more stuff into an existing test

    • -4
    • +13
    ./tcl/test/acs-authentication-procs.tcl
Test for auth::get_user_id and the whole login attempts api

    • -0
    • +167
    ./tcl/test/authentication-procs.tcl
Document actual behavior

    • -2
    • +3
    ./tcl/authentication-procs-aolserver.tcl
    • -2
    • +3
    ./tcl/authentication-procs-naviserver.tcl
Test auth::get_user_id

    • -0
    • +53
    ./tcl/test/authentication-procs.tcl
Test auth::get_register_authority proc

    • -0
    • +49
    ./tcl/test/authentication-procs.tcl
Fix test name

Test auth::get_all_registration_elements

    • -0
    • +21
    ./tcl/test/authentication-procs.tcl
Test more public api

    • -0
    • +47
    ./tcl/test/authentication-procs.tcl
Test auth::can_admin_system_without_authority_p public api

    • -0
    • +103
    ./tcl/test/authentication-procs.tcl
file authentication-procs.tcl was initially added on branch oacs-5-10.

    • -0
    • +0
    ./tcl/test/authentication-procs.tcl
Test auth::authority::get_sc_impl_columns public api

file authority-procs.tcl was initially added on branch oacs-5-10.

    • -0
    • +0
    ./tcl/test/authority-procs.tcl
Reduce divergency between oracle and postgres codebase

    • -19
    • +0
    ./tcl/test/acs-authentication-procs-oracle.xql
    • -19
    • +0
    ./tcl/test/acs-authentication-procs-postgresql.xql
    • -2
    • +11
    ./tcl/test/acs-authentication-procs.tcl
  1. … 6 more files in changeset.
fix release dates in .info files

  1. … 82 more files in changeset.
release work

  1. … 204 more files in changeset.
regenerated documentation

  1. … 467 more files in changeset.
bump version numbers of 5.9|5.10 packages to 5.10.0b1

adjust dependencies

  1. … 87 more files in changeset.
improve spelling

    • -2
    • +2
    ./tcl/authentication-procs-naviserver.tcl
  1. … 42 more files in changeset.
Bring the implementation of the "Search" operation of the "auth_search" service contract in line with the operation's definition by correcting the returned value ("username" instead of "user_id") and restricting the search to the local authority.

Streamline idiom: we are only interested in the user ids

Comment deprecated proc, which was before defined as private, because of a false positive in acs-api-browser.callgraph__bad_library_calls: the "Authenticate" operation in the spec of auth::authentication::create_contract is mistakenly interpreted as the deprecated proc

Make proc public, as was intended

Deprecated trivial private ad_user_login wrapper, which is already a public api

Promote logics in auth::authentication::Authenticate to a public api, which unfortunately cannot be the same due to the inconsistent naming, hence we have to create a new alias

Rationale:

- the proc provides some value to the user by choosing the right contract implementation depending on the authority

- we found occurrences in other packages downstream, and as internally this proc is calling only public api, one could just duplicate the whole thing and be compliant with our standards, which is not an improvement IMO

- the proc was already tested and documented

    • -4
    • +4
    ./tcl/test/acs-authentication-procs.tcl
  1. … 1 more file in changeset.
Mark 'auth::can_admin_system_without_authority_p' as public, as it is used in 'acs-admin'

Mark 'auth::authority::get_sc_impl_columns' as public, as it is used in 'acs-admin'

Make 'auth::get_register_authority' public, as it is used by acs-subsite

Make 'auth::verify_account_status' public, as it is used in other packages. Fixes 'callgraph__bad_page_calls' test case

Use the new 'url' and 'email' input type widgets

Port of downstream logic: do not use a hardcoded list of local authorities, but rather check if the authority implementation is local, which would handle also other downstream local authorities

Rework the idiom to perform cheap checks first and improve documentation