• last updated 4 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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

Fix var name

Creating a user without a username is possible if the authority is local, include the test authority among those for which we generate a username, as it is also local as in "hosted locally"

Fixes acs-authentication.auth_use_email_for_login_p automated test

reduce public footprint

  1. … 1 more file in changeset.
fixed bug: exception in test case auth_use_email_for_login_p caused test to terminate

By adding an "ad_try", we can see that the intention to create

a user with no username failed. Probably this was working at some time,

not sure, if it still should.

This is could be a general problem with "aa_run_with_teardown" to

terminate too early in such cases.

    • -10
    • +18
    ./tcl/test/acs-authentication-procs.tcl
- Rework acs-api-browser.callgraph__bad_calls automated test so that acs-service-contract calling contract implementation is considered OK

- Reinstate authentication service contracts as private

- Move test authority implementation to acs-authentication to remove intra-package dependency

acs-api-browser.callgraph__bad_call still fails because proc acs-tcl.install::xml::action::create-user calls private acs-authentication.auth::local::registration::Register... to be continued

  1. … 2 more files in changeset.
file test-authority-procs.tcl was initially added on branch oacs-5-10.

    • -0
    • +0
    ./tcl/test-authority-procs.tcl
Make api public, complies with acs-api-browser.graph__bad_calls automated test

  1. … 1 more file in changeset.
Make service contract implementation public

Complies with acs-api-browser.callgraph__bad_calls

improve listing of test coverage

    • -1
    • +27
    ./tcl/test/acs-authentication-procs.tcl
  1. … 16 more files in changeset.
make listing of tested procs more complete

    • -12
    • +30
    ./tcl/test/acs-authentication-procs.tcl
  1. … 17 more files in changeset.
whitespace changes

set default value only in case of failure

    • -6
    • +7
    ./tcl/authentication-procs-naviserver.tcl
Care about the parameter cache after test case

    • -1
    • +3
    ./tcl/test/acs-authentication-procs.tcl
Set a bunch of test cases to 'production_safe'

    • -7
    • +7
    ./tcl/test/acs-authentication-procs.tcl
Merge 'auth_authority_api' and 'auth_authority_delete' test cases, covering almost the same api

    • -195
    • +122
    ./tcl/test/acs-authentication-procs.tcl
Fix typo

    • -2
    • +2
    ./tcl/test/acs-authentication-procs.tcl
don't rely on uppercase header field names

  1. … 10 more files in changeset.
prefer db_0or1row +"where exists" over db_string + "select case"

  1. … 3 more files in changeset.
Improve sql portability: oracle cannot select a boolean directly using exists

Reduce divergencies between oracle and postgres codebase

    • -9
    • +0
    ./tcl/test/acs-authentication-procs-oracle.xql
    • -10
    • +0
    ./tcl/test/acs-authentication-procs-postgresql.xql
    • -2
    • +6
    ./tcl/test/acs-authentication-procs.tcl
prefer "ns_conn isconnected" over "ad_conn isconnected", since the latter might not notice changes in the connect state

  1. … 18 more files in changeset.
mprove spelling: move closer to the linux documentation recommendations

    • -2
    • +2
    ./tcl/test/acs-authentication-procs.tcl
  1. … 15 more files in changeset.
Fallback to + #

Add missing authority_id on the 'auth_authenticate' testcase

    • -1
    • +2
    ./tcl/test/acs-authentication-procs.tcl
Replace deprecated idiom

break overlong lines

Replace deprecated idiom