• last updated 10 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Make also sure the tmpfile from the widget exists beforehand, when validating: we don't want users to "explore" our tmpdir with bogus values that just look sane

  1. … 1 more file in changeset.
Improve doc

Revert to previous template::widget::file behavior of accepting input in a form of a list of 3 elements (e.g. without a .tmpfile in the request), but introduce validation so that we enforce all widget values to be in the proper format and the files to be "safe"

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

Test the behavior of the file-storage when a malicious user would try to store a pre-existing file on the server as its own

The fix for the file-storage is a simple validation to make sure that the tmpfile exists, however, for the generic case of the file widget, we cannot trust the tmpfile value when this was not generated by the server. This will probably cause regression when one wants to show a "preview" of a form, to be continued.

  1. … 2 more files in changeset.
Incorporate feedback from downstream penetration tests:

do not skip basic validation in the file widget when a tmpfile is not found in the request.

extended adp:icon handling

Added support for multiple classes,

Added more potentially reusabel icon names,

bumped version to 5.10.1d13

  1. … 1 more file in changeset.
extended generic names by "user" and "list-alt"

whitespace changes

added new generic icon name: add-new-item

prefer parameter::get_from_package_key over fetching package_id first

fix false positive of regression test

removed code factored out to the separate "captcha" package

  1. … 10 more files in changeset.
improve comments, such that is is clear what command to use instead

deactivate captcha test for the time being

Constrain captcha width to 100% of its parent's size

whitespace changes

reduce useless warnings for template::adp_parse_tags

The problem was that template::adp_compile_chunk produces no output, when the parsed chunk contains only whitespace,

causing an error message in this case. The changed method shortcuts the behavior.

Fix missing variable

Fix typo

Provide an alternative exec-free captcha implementation using libgd and its Tcl wrapper https://github.com/flightaware/tcl.gd

This implementation will be preferred to the one using convert, when available.

  1. … 1 more file in changeset.
Cleanup dummy commands and objects to avoid cross interactions between tests

  1. … 1 more file in changeset.
Go back supporting arbitrary options, potentially from custom user-defined widgets: these will be assumed to be expressed as name-value pairs and we will keep notifying such cases in the logfile

whitespace changes

revive delete regression test (was formerly in xowiki)

Keep supporting also the htmlarea_p option

Fix doc

Keep handling also the section option

Keep recognizing "format" template::element property, which was never documented

completed generic icon tables