Skip to content

Devel/directory handling rebased and merged 260718 - #360

Open
joakimlindblad wants to merge 25 commits into
developfrom
devel/directory_handling_rebased_and_merged_260718
Open

Devel/directory handling rebased and merged 260718#360
joakimlindblad wants to merge 25 commits into
developfrom
devel/directory_handling_rebased_and_merged_260718

Conversation

@joakimlindblad

Copy link
Copy Markdown
Member

Allow subdirectories in the image data folder.
Should work with history as well (directory encoded in image name, e.g., "image": "dir1/Hamamatsu-OS-2").
Sanitizing on server side, to avoid any data leakage (by stuff like folder='../..').
New URL option: folder=dir, to open image browser at specific folder (overruled by image, if both given).

Develop is not rebased on this branch (due to too many problems) but merged.

Important change: input sanitazion is primarily moved to cytobrowser.js (as early as possible)
I hope this brings the 'deve/directory_handling' branch up to date
@joakimlindblad joakimlindblad self-assigned this Jul 18, 2026
@joakimlindblad
joakimlindblad requested a review from Olleesch July 18, 2026 14:07
@joakimlindblad
joakimlindblad marked this pull request as draft July 18, 2026 14:09
@joakimlindblad
joakimlindblad marked this pull request as ready for review July 18, 2026 14:10
Olleesch and others added 2 commits August 5, 2026 19:57
…f there are no directories before giving a 'noavailableimages' error message
…risk of race issues with many users.

ctime-check to rescan the directory back in use (disappeared in commit b12c575 (or rather 23e0e58) for unknow reason).
@joakimlindblad

Copy link
Copy Markdown
Member Author
  • activePath and activeDir variables no longer shared across the module, but the path is bundled with the availableImages as a unit; should hopefully reduce risk of race issues.
  • ctime check reinstated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants