OpenACS / openacs-4 / packages / acs-tcl

tcl

Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 71
Last Commit: 07 May
Commits this week: 0
Total Lines of Code (LoC): 65,995
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated 16 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Monday 29 Aug 2022
3:35 pm

antoniop

Fix typo

Options
    • -2
    • +2
    ./text-html-procs.tcl
3:02 pm

gustafn

implemented ad_html_security_check based on ns_parsehtml

Options
    • -44
    • +144
    ./text-html-procs.tcl
1:53 pm

gustafn

Use "ns_parsehtml" for speed and correctness when available

The new NaviServer built-in command "ns_parsehtml" is used on

several occasions in the code when available, namely in

- util_close_html_tags

- escaping disallowed HTML tags in ad_text_to_html

- util_close_html_tags

The command "util_close_html_tags" has a long history of troubles,

since it was implemented originally based on regular expressions. It

was reported in Jan 2009, that the function took on certain forums

entries 6 to 9 hours (e.g. /forums/message-view?message_id=357753) to

finish. Therefore, at this time a quick version based on tdom was

introduced, which reduced at that time the performance to 7.3

seconds. However, the tdom-based version tend to fail on invalid HTML,

so it could cover only a few cases. The new "ns_parsehtml" is more

robust and works very stable on openacs.org with the know problem

cases. Furthermore, this version is also significantly faster than the

tdom version.

Options
    • -33
    • +269
    ./text-html-procs.tcl
    • -6
    • +17
    ./test/html-conversion-procs.tcl
12:34 pm

gustafn

improve spelling

Options
    • -3
    • +3
    ./deprecated-procs.tcl
    • -2
    • +2
    ./security-procs.tcl
    • -3
    • +3
    ./tcl-documentation-procs.tcl
    • -1
    • +1
    ./test/acs-db-00-procs.tcl
    • -1
    • +1
    ./test/apm-install-procs.tcl
    • -10
    • +69
    ./test/html-conversion-procs.tcl
    • -7
    • +7
    ./test/object-test-case-procs.tcl
11:47 am

antoniop

Fix typo

Options
    • -1
    • +1
    ./test/doc-check-procs.tcl
11:06 am

gustafn

fix bug in db_multirow_group_last_row_p

The bug showed up in bug-tracker usage of multirows,

where db_multirow_group_last_row_p was still expecting the

dict structure

Options
    • -3
    • +10
    ./01-database-procs.tcl
Saturday 27 Aug 2022
10:35 pm

gustafn

use icanuse "ns_parsehtml" and new interface

Options
    • -5
    • +4
    ./test/doc-check-procs.tcl
10:34 pm

gustafn

added icanuse rule for ns_parsehtml

Options
    • -0
    • +1
    ./00-icanuse-procs.tcl
10:30 pm

gustafn

Give people the chance to use OpenACS with WithDeprecatedCode set to 0

When OpenACS is configured with loading of deprecated procs

deactivated, files like deprecated-procs.tcl are not

loaded. Therefore, these files should only contain code, which was

deprecated at LEAST ONE RELEASE EARLIER, such that site admins have one

release time to fix calls to deprecated code.

This change reverts in part Antonio's commit from 26 Aug 2022 which

causes errors like the following on openacs.org:

Error in include template "/var/www/openacs.org/packages/news/lib/news":

invalid command name "ad_parameter_all_values_as_list"

Options
    • -4
    • +24
    ./defs-procs.tcl
    • -22
    • +10
    ./deprecated-procs.tcl
Friday 26 Aug 2022
3:00 pm

antoniop

Deprecate ad_parameter_all_values_as_list: it does not provide much value and was also often use improperly

Options
    • -2
    • +2
    ./apm-file-procs.tcl
    • -15
    • +1
    ./defs-procs.tcl
    • -2
    • +21
    ./deprecated-procs.tcl
    • -30
    • +9
    ./text-html-procs.tcl
    • -2
    • +0
    ./test/html-conversion-procs.tcl
2:06 pm

antoniop

tmpfile page contract filter reform:

do not allow acs-subsite TmpDir parameter to define where the tmpfolder is located anymore. This MUST be the one configured in the server-wide configuration. Tmpfiles cannot be in a subfolder of the tmpfolder, they MUST be direct children instead. A tmpfile MUST exist beforehand and be owned, be readable and writable by the user running the nsd process. This complies with the definition of a tmpfile by AolServer/NaviServer when they are created to store content coming from a file upload.

Options
    • -36
    • +2
    ./security-procs.tcl
    • -24
    • +10
    ./tcl-documentation-procs.tcl
    • -28
    • +0
    ./test/security-procs.tcl
  4. … 2 more files in changeset.
Thursday 25 Aug 2022
6:01 pm

antoniop

Extend tmpfile filter to behave in the "old way" (default), or in strict mode e.g. tmpfile(strict), enforcing the behavior for tmpfile in Aolserver/Naviserver when a form is processed

Options
    • -12
    • +30
    ./tcl-documentation-procs.tcl
5:37 pm

antoniop

Reimplement ad_page_contract_filter_proc_tmpfile using security::safe_tmpfile_p

Some of the features implemented by this filter have been ported into the api, namely the possibility to fetch the valid temp folders from the subsite TmpDir parameter and the possibility to relax the check and allow also files deeper in the tmpfolder hierachy.

Notably, the hardcoded tmpfolders "/var/tmp" and "/tmp" have NOT been ported. One should configure these values via the many available options. security::safe_tmpfile_p is also more restrictive when a file exists, because it checks for ownership and read and write permissions on the file.

Options
    • -3
    • +42
    ./security-procs.tcl
    • -17
    • +8
    ./tcl-documentation-procs.tcl
    • -1
    • +30
    ./test/security-procs.tcl
4:16 pm

antoniop

Comment WIP

Options
    • -6
    • +6
    ./parameter-procs.tcl
3:12 pm

antoniop

Declare proc coverage

Options
    • -0
    • +2
    ./test/http-client-procs.tcl
  2. … 1 more file in changeset.
3:09 pm

antoniop

Factor the payload building behavior in util::http::post into its own proc, so that it can be reused elsewhere

Options
    • -69
    • +170
    ./http-client-procs.tcl
2:37 pm

antoniop

Add must_exist flag to enforce a safe tmpfile to already exist

Options
    • -4
    • +9
    ./security-procs.tcl
    • -1
    • +6
    ./test/security-procs.tcl
  3. … 1 more file in changeset.
1:44 pm

gustafn

Fixed bug in util_convert_line_breaks_to_html

The code deleted spaces around certain tags, while - according to the

documentation, only line breaks should be removed from there.

Extended regression test.

Options
    • -4
    • +5
    ./text-html-procs.tcl
    • -5
    • +14
    ./test/html-conversion-procs.tcl
1:25 pm

antoniop

Introduce security::safe_tmpfile_p checking whether a file belongs to the configured tmpfolder and respects other constraints

The plan is to use it to improve input validations

Options
    • -1
    • +54
    ./security-procs.tcl
    • -0
    • +45
    ./test/security-procs.tcl
  3. … 1 more file in changeset.
Wednesday 24 Aug 2022
4:51 pm

gustafn

Deactivated script-dynamic again

activating "script-dynamic" for script-src prevents resources as the following from loading

<script type="text/javascript" src="/resources/xowiki/bootstrap-treeview/1.2.0/bootstrap-treeview.min.js" nonce="F8843D5CD542FB1CDB4C94C5D13C4E5ECC2E1DCA">

Options
    • -2
    • +2
    ./security-procs.tcl
4:30 pm

gustafn

improve test coverage of apm_* API

Options
    • -0
    • +24
    ./test/apm-procs.tcl
2:13 pm

gustafn

added test for checking documentation in regard of valid HTML markup

Options
    • -2
    • +53
    ./test/doc-check-procs.tcl
Tuesday 23 Aug 2022
8:44 pm

gustafn

Properly escape "<" and ">" in api-doc documentation.

Since all documentation is rendered via HTML, the characters

"<" and ">" have to be HTML-quoted, otherwise strange things

(omission, unintended renderings) might occur.

E.g. the sentence

Define an interface between a page and an

ADP <include> similar to the page_contract.

was rendered as

Define an interface between a page and an

ADP similar to the page_contract.

which is incorrect.

Options
    • -6
    • +7
    ./deprecated-procs.tcl
    • -5
    • +9
    ./security-procs.tcl
    • -17
    • +22
    ./tcl-documentation-procs.tcl
    • -7
    • +6
    ./utilities-procs.tcl
  5. … 10 more files in changeset.
1:10 pm

gustafn

Changed "db_multirow" to use "db_list_of_lists" instead of "db_list_of_ns_sets"

Thanks to the recent improvements in NaviServer for

"db_list_of_lists", it is more efficient to use this in "db_multirow"

(actually in "db_multirow_helper"). The implementation revealed a bug

in "db_list_of_lists", when both options "-columns_var" and

"-with_headers" were set.

Extended regression test to cover the former bug.

Options
    • -9
    • +19
    ./01-database-procs.tcl
    • -1
    • +17
    ./test/db-proc-test-procs.tcl
Friday 19 Aug 2022
12:16 pm

antoniop

Relax tdom tolerance for non-standard markup

Options
    • -2
    • +3
    ./apm-install-procs.tcl
12:15 pm

antoniop

Test apm_get_repository_channels proc

Options
    • -0
    • +25
    ./test/apm-install-procs.tcl
12:15 pm

antoniop

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

Options
    • -0
    • +0
    ./test/apm-install-procs.tcl
Wednesday 17 Aug 2022
12:43 pm

antoniop

Fix typo

Options
    • -1
    • +1
    ./test/db-proc-test-procs.tcl
Thursday 11 Aug 2022
3:49 pm

gustafn

Generalized "version_dir" handling a little for download specs

The problem was that bootstrap5 uses a version directory, which

consists of the version plus an extra string element. The previous

version assumed that the version is always used as a directory name.

Whe track now the versionDir information in the resource_info dict and

use this, when available (otherwise the version number is used as before).

The resource_info dict contains now the following path components:

# Provide paths for loading either via /resources/ or CDN

#

# "resourceDir" is the absolute path in the filesystem

# "resourceUrl" is the URL path provided to the request processor

# "versionDir" is the version-specific element both in the

# URL and in the filesystem.

#

bumped acs-tcl to 5.10.1d19

Options
    • -1
    • +27
    ./utilities-procs.tcl
  2. … 2 more files in changeset.
Monday 08 Aug 2022
4:48 pm

gustafn

fix several bugs in "acs_object_type::supertype" and improve code sanity

Options
    • -8
    • +8
    ./object-type-procs-postgresql.xql
    • -8
    • +8
    ./object-type-procs.tcl
    • -1
    • +10
    ./test/object-test-case-procs.tcl