• last updated 12 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
reduce verbosity

align nameing with 'resource_info_procs'

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

claim test coverage

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

removed progress bar from curl output

wget is not used anymore

silence regression test run

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

  1. … 1 more file in changeset.
fixed resource leak

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

More resource-info updates:

- fixed wrong and inconsistent naming of dict members (many thanks to Sebastian Scheder for figuring this out)

- removed duplicated slashes in resource paths

- fixed incorrect paths when CDN is used

- simplified handling of cspMaps

- added test checking consistency of resource-info dicts

  1. … 9 more files in changeset.
file resource-info-procs.tcl was initially added on branch oacs-5-10.

    • -0
    • +0
    ./test/resource-info-procs.tcl
fix for regression test case ad_context_bar_multirow

originally, site-node entries survived the test

after the first fix, the regression test was failing (sometimes?) the complaint was about a missing package.

now, the test seems to work always correctyl, and the site-nodes entries are gone as well

Fix regression in db_multirow_group_last_row_p

Row is last when the next value changes

Many thanks to Miachel Aram for reporting the issue