diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-16 11:30:04 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-16 11:30:04 -0500 |
| commit | ab7a557a5941723d0b3bd7f639efc414cc5754fd (patch) | |
| tree | df1af41c00153f78603578dcceb18e292618006c /tests/test-auth-config-toggle-auth-source-debug.el | |
| parent | 0996d2fc71e115dc895fbf30eb41dbf09fda5962 (diff) | |
| download | dotemacs-ab7a557a5941723d0b3bd7f639efc414cc5754fd.tar.gz dotemacs-ab7a557a5941723d0b3bd7f639efc414cc5754fd.zip | |
feat(gptel-tools): harden path validation with file-truename realpath
Resolves PATH through file-truename before applying home-directory and
read/write checks across the path-handling tools (git_status, git_log,
git_diff, move_to_trash, read_text_file, update_text_file,
write_text_file, list_directory_files, read_buffer, web_fetch).
Without the resolve step, a symlink under HOME pointing outside HOME
would pass the prefix check but the tool would act on the real target
-- a symlink-escape.
move_to_trash also tightens the trash-bin construction (treats empty
file extensions correctly) and switches the "critical directories"
list to truename-resolved canonical forms so a symlinked ~/.config
can't be trashed via an aliased path.
update_text_file fixes an off-by-one in the line-count derivation
when the source content is empty.
Each source change pairs with tests in tests/test-gptel-tools-*.el
and tests/test-update-text-file.el covering the realpath escape
paths, the empty-extension trash case, and the empty-content line-
count edge. Combined coverage is now 100% across all ten gptel-tools
source files: 516 / 516 executable lines, 217 tests.
Diffstat (limited to 'tests/test-auth-config-toggle-auth-source-debug.el')
0 files changed, 0 insertions, 0 deletions
