• last updated 16 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Prefer ad_mktmpdir to ad_tmpnam

merge with missing files

    • -0
    • +0
    ./sql/oracle/file-storage-views-create.sql
  1. … 1450 more files in changeset.
merge from oacs-5-10

    • -0
    • +2
    ./catalog/file-storage.de_DE.ISO-8859-1.xml
    • -2
    • +2
    ./catalog/file-storage.en_AU.ISO-8859-1.xml
    • -17
    • +19
    ./catalog/file-storage.en_US.ISO-8859-1.xml
    • -3
    • +3
    ./catalog/file-storage.es_CO.ISO-8859-1.xml
    • -3
    • +5
    ./catalog/file-storage.es_ES.ISO-8859-1.xml
    • -3
    • +3
    ./catalog/file-storage.es_GT.ISO-8859-1.xml
    • -0
    • +2
    ./catalog/file-storage.it_IT.ISO-8859-1.xml
    • -6
    • +152
    ./sql/oracle/file-storage-package-create.sql
    • -6
    • +3
    ./sql/oracle/file-storage-views-create.sql
    • -0
    • +49
    ./sql/oracle/upgrade/upgrade-5.10.d2-5.10.d3.sql
  1. … 8085 more files in changeset.
bumped version number from development version to release version

  1. … 93 more files in changeset.
regenerated documentation

  1. … 438 more files in changeset.
improved spelling

  1. … 15 more files in changeset.
silence expected errors during regression test

    • -10
    • +16
    ./tcl/test/file-storage-procs.tcl
made regression test more robust

The test was failing, when the preferred language in the browser

was not "en". The test is now slightly better, but one should

use CSS classes (or data attributes) to pick the bulk actions

regenreated documentation (with non-finished release notes)

  1. … 146 more files in changeset.
fixed typoes in documentation

  1. … 23 more files in changeset.
improved spelling

  1. … 29 more files in changeset.
Harden page contract

Harden page contract

  1. … 2 more files in changeset.
Remove additional trapping of exception, which prevents the web agent to access the file-storage, if we had to mount it on the fly

    • -30
    • +30
    ./tcl/test/file-storage-procs.tcl
improved test coverage

  1. … 1 more file in changeset.
Document more explicitly that we do not perform filesystem operations when deleting files via the fs:: api, but we rely on the content repository and its trigger+scheduled proc mechanism

download-archive reform

File-Storage used to generate downloaded archives in tgz format, to then switch to zip, more user-friendly, in particular outside the Linux world (See https://openacs.org/forums/message-view?message_id=557561). To ease the transition, a couple of parameters and relative API were introduced that would allow to choose the preferred command one should use. During this reform however, default parameter values in the tcl code became inconsistent with those in the info file. Furthermore, the chosen defaults were set as absolute paths to the executable, which is not friendly to non-linux environments, or other scenarios where the "typical" Linux filesystem structure cannot be assumed (e.g. containers, MacOS...).

The only usage of this parameters/api was in fact in the download-archive vuh. In upstream codebase, no package references this file, not even the file-storage itself. Upon review, one could see that the file would also allow to specify a custom download filename via the path, which could be considered questionable. It would also execute the command in a way that once again assumes some form of Linux environment (e.g. invoking bash).

Save for the ability to customize the archive format and the anti-feature of being able to manipulate the archive filename via the path, the script largely relplicates www/download-zip, in a better shape after a few reforms hinted by e.g. penetration tools.

Given the aformentioned considerations, I have decided to make download-archive a simple redirect to download-zip. Specifying the object_id via the path will keep working, while URLs out there expecting the name to change will not fail, but the name will not be modified. The archive format will from now on be assumed to be zip.

    • -1
    • +15
    ./tcl/file-storage-install-procs.tcl
    • -57
    • +24
    ./www/download-archive/index.vuh
Cleanup of external binaries: always use "util::which" to resolve binaries

  1. … 5 more files in changeset.
Improved consistency with external programs

Since "unzip" is used as well on various other places,

use it as well in the file storage. This means that

the parameter "UnzipBinary" for the file-storage package

is now obsolete.

strengthen boolean parameters in page contracts

  1. … 23 more files in changeset.
Make values optional, as the user should supply them via the form

Reintroduce "short_name" list element in the folder-chunk, used in the list format and not exactly equivalent to "name"

bumped version numbers to 5.10.1b1

  1. … 85 more files in changeset.
regenerated documentation

  1. … 294 more files in changeset.
Sign the return_url we set when we generate a zip file to prevent tampering

Test external command dependency for the package

Make sure, that also when the number of days is supplied as empty, we set it to a valid default

harden page_contract

Fix english

Deprecate fs::get_folder_contents