• last updated 9 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
reduce verbosity during regression test

do not complain about expected behavior

silence regression test

don't expect that invalid input becomes magically valid

reduce verbosity

align nameing with 'resource_info_procs'

  1. … 7 more files in changeset.
silence test cases

claim test coverage

    • -1
    • +3
    ./tcl/test/community-core-test-procs.tcl
fixed old bug via regression test in page contract filters

The binding variable for "min_length" and others was not passed correctly.

We saw unexpected warning of the following form during regression test:

Warning: Message contains a variable named 'min_length' which doesn't exist in the caller's environment:

handle cases in the page contract, when not connected (used this way in the regression test)

keep log file viewable (don't add too many entries)

remove leftover files after run, silence regression test run

whitespace changes

  1. … 10 more files in changeset.
silence test cases

    • -3
    • +5
    ./tcl/test/community-core-test-procs.tcl
removed progress bar from curl output

wget is not used anymore

silence regression test run

    • -36
    • +59
    ./tcl/test/db-proc-test-procs.tcl
comment unexpectged behaviour

Silence several log entries during initial install and during boot.

The system should complain about missing caches just, when

a) not during initial install, and

b) if this is not during startup of an installed version

removed potentially misleading dot in dependency message

make query name fully qualified to remove warning

Fix potential problems when calling polymorphic SQL functions from Tcl

Some functions are defined in the database with the same number of

arguments but different types, e.g., first argument "package_key"

(type text) or "package_id" (type integer). This is fine from the SQL

standpoint, but when calling from Tcl via bind-vars

(e.g. ":package_id"), everything is passed as a string, and

potentially, the wrong function is called.

Now, all the automatically generated subs are generated with casts,

when the integer based variant must be called.

Some examples:

Before:

set s [ns_pg_bind 0or1row $__DB {select apm__set_value(:package_id,:parameter_name,:attr_value)}]

set s [ns_pg_bind 0or1row $__DB {select apm__get_value(:package_id,:parameter_name)}]

Now:

set s [ns_pg_bind 0or1row $__DB {select apm__set_value(CAST(:package_id AS integer),:parameter_name,:attr_value)}]

set s [ns_pg_bind 0or1row $__DB {select apm__get_value(CAST(:package_id AS integer),:parameter_name)}]

- bumped version number to 5.10.1b11

fixed resource leak

    • -45
    • +45
    ./tcl/test/acs-tcl-test-procs.tcl
reduced verbosity

Make localurl filter more robust to invalid URLs: fail validation in this case, intead of crashing

fixed resource leaks

dom trees were not deleted

ad_dom_sanitize_html: fixed 2 resource leaks

in case of parsing errors in the input string the following structures leaked:

- dom tree

- stuct::tree

make sure to delete tdom document

Delete nodes explicitly, otherwise it survives the end of the proc.

fixed icanuse-test for deleting samesite cookies