Feat/s3 file browser v1 by dklOrdix · Pull Request #208 · stackabletech/cockpit

dklOrdix · 2026-06-18T11:08:13Z

Closes #129

…in +layout.svelte

…ebar imports Co-authored-by: Copilot <copilot@github.com>

Co-authored-by: Copilot <copilot@github.com>

…s/utils Co-authored-by: Copilot <copilot@github.com>

Co-authored-by: Copilot <copilot@github.com>

feat(storage): initial Filebrowser UI implementation

Co-authored-by: Copilot <copilot@github.com>

…ation S3 File Browser Page Navigation (Pagination)

* #96 disable the storage browser if feature flag is set * #96 fix svelte-check * #96 fix lint * #96 disable storage browser by default

* #102 refactor pagination controls into a separate component * #102 simplified pagination props

* #93 add modal preview for file display * #93 fix broken e2e tests * #93 add preview for parquet files, fix scrollbar visibility in preview, allow ms excel csv data for preview * #93 improve parquet row fetch handling * #93 fix lint * #93 move preview endpoint logic into separate files * #93 disable parquet preview

* refactor(ObjectTable): extract File and Folde rows into seperate components * style: format code * style: use camelCase * style: use camelCase

* feat: implement single-file download functionality and error handling for S3 storage * feat: refactor S3 download endpoint to use new API structure and improve error handling * chore: remove obsolete note about static download route segment in tech debt documentation * style: format code * fix: improve type safety in S3 error handling * fix: preview action * feat: add context menu header and improve file selection handling in FileExplorer * fix: reset selection state on context menu close and refine download action logic * style: format files * style: remove pointer events from Actions row on ContextMenu * #91 fix lint --------- Co-authored-by: F3l1x1vo <fli@ordix.de>

Bohreromir · 2026-09-18T08:16:13Z

+/**
+ * Shared helpers for architecture fitness tests.
+ *
+ * ArchUnitTS only scans TypeScript (.ts) source files. Svelte (.svelte) and


Bohreromir

please do not mark my comments as resolved, I loose track of them otherwise

Bohreromir · 2026-09-22T12:27:47Z

F3l1x1vo · 2026-09-23T07:30:02Z

@Bohreromir I've implemented most of your suggestions in 9e2ec68. I have two points to address:

When creating a new file paste (ctrl+p) does not work in the filename field

Ctrl + P is not a valid shortcut. File paste works on Ctrl + V.

Fonts look very faint/smeared/weird

It's the regular bold disabled font we use everywhere. I don't disagree that it appears slightly blurred if you look closely in darkmode but it's clearly legible. This was accepted from the day we first did a mockup with the Stackable Team of the storage browser.

Bohreromir · 2026-09-23T09:10:51Z

@F3l1x1vo Whoops my bad. I meant using ctrl+v into the field for the new file name

F3l1x1vo · 2026-09-23T10:52:57Z

I meant using ctrl+v into the field for the new file name

There's a conflict there with the regular pasting of text. If the user wants to copy and paste a filename he copied from outside the browser for example, a file in the clipboard of the storage browser would be pasted into that field instead of his text. This pasting of filenames by the clipboard is a very niche and weird interaction that I would suggest to leave out.

Bohreromir

some more

Bohreromir · 2026-09-23T13:10:18Z

+    return { entries, hasMore: false };
+  } catch (err) {
+    throw new Error(
+      'RAR support requires the "unrar" command to be installed on the server. ' +


Aren't we the ones controlling the server?

Apart from that the unrar license says:
"UnRAR source code may be used in any software to handle
RAR archives without limitations free of charge, but cannot be
used to develop RAR (WinRAR) compatible archiver and to
re-create RAR compression algorithm, which is proprietary.
Distribution of modified UnRAR source code in separate form
or as a part of other software is permitted, provided that
full text of this paragraph, starting from "UnRAR source code"
words, is included in license, or in documentation if license
is not available, and in source code comments of resulting package."
https://github.com/pmachapman/unrar/blob/master/license.txt

I dont speak lawyer, but that might be something @lfrancke wants to check.

Honestly I'd drop the whole archive thing.

As is the pod sizing limits raw archive size and extracted size together to 120MiB. This doesn't sound a lot to me, especially when shared between users.

Apart from that rar extraction is broken as of now, even with unrar present.
execFileAsync has no encoding option, so stdout is UTF-8-decoded into a string; Buffer.from(stdout, 'binary') (archive.ts:574) then takes the low byte of each char. Any byte ≥ 0x80 becomes U+FFFD and comes back as 0xFD. Works for Text/ASCII, but Parquet inside RARs failes

Seems like we have to remove 7zip and rar extraction completely.

Bohreromir · 2026-09-23T13:11:52Z

+  maxBytes?: number
+): Promise<ArchiveListing> {
+  const bin = await find7zBinary();
+  if (!bin) throw new Error('7z support requires 7-Zip to be installed on the server.');


where is this installed? including it in the build has the same rar problem described above as it includes the rar archiever

Bohreromir · 2026-09-28T09:27:29Z

+  const raw = env.STORAGE_ENCRYPTION_KEY;
+
+  if (!raw) {
+    log.error('STORAGE_ENCRYPTION_KEY environment variable is not set');


this .env is missing in .example.env

Resolved in 1468557.

Bohreromir · 2026-09-28T10:24:29Z

+    return { entries, hasMore: false };
+  } catch (err) {
+    throw new Error(
+      'RAR support requires the "unrar" command to be installed on the server. ' +


Seems like we have to remove 7zip and rar extraction completely.

Bohreromir · 2026-09-28T10:26:03Z

+  }
+}
+
+// ── 7z ───────────────────────────────────────────────────────────────────────


please remove 7zip support as we can not ship the requried binary

Bohreromir · 2026-09-28T10:26:17Z

+  });
+}
+
+// ── RAR ──────────────────────────────────────────────────────────────────────


please remove rar support as we can not ship the required binary

dklOrdix and others added 30 commits May 4, 2026 09:18

refactor(frontend): clean up and restructure app

161ce0e

fix: add lint:fix script to package.json and clean up trailing comma …

d2de208

…in +layout.svelte

docs: update AGENTS.md to include Material UI icon library usage

c81b4d0

refactor(header): move components for header into own directory

9add069

chore: dedupe lockfile

8343022

fix(sidebar): add hover cursor style to toggle button

4bc8e93

chore(types): centralise navigation and auth types; update header/sid…

952dc80

…ebar imports Co-authored-by: Copilot <copilot@github.com>

feat(storage): initial Filebrowser UI implementation

cba7866

Co-authored-by: Copilot <copilot@github.com>

refactor(storage): extract service layer and consolidate storage type…

5de8d9f

…s/utils Co-authored-by: Copilot <copilot@github.com>

Merge branch 'feat/s3-file-browser' into feat/s3-file-browser-ui

20909ae

chore(storage): remove outdated STORAGE_REORG_PLAN.md

2b33533

fix(lint): resolve errors

aeac85d

refactor(tests): update ThemeToggle tests to use checkbox role

f2ba284

Co-authored-by: Copilot <copilot@github.com>

style: format code

af9b651

fix(tests): update theme toggle role from button to checkbox

8373974

Co-authored-by: Copilot <copilot@github.com>

fix: revert changes to Theme Toggle

7aff59b

fix: revert changes to Theme Toggle

266a045

Merge pull request #90 from stackabletech/feat/s3-file-browser-ui

805f693

feat(storage): initial Filebrowser UI implementation

#80 use pagination controls to navigate the file browser

3386182

#80 show page counter for pagination control

b77bcfa

Co-authored-by: Copilot <copilot@github.com>

#80 fix lint and svelte check

48b552e

Co-authored-by: Copilot <copilot@github.com>

fix lint error

7c9733f

Merge pull request #100 from stackabletech/feat/s3-file-browser-pagin…

9d85f57

…ation S3 File Browser Page Navigation (Pagination)

File Browser: Feature Flag (#105)

b4e25ec

* #96 disable the storage browser if feature flag is set * #96 fix svelte-check * #96 fix lint * #96 disable storage browser by default

Rename option STACKABLE_UI_COMPLETION_DISABLED (#106)

c36f1c8

S3 File Browser: Shared Pagination Component (#103)

ff43b22

* #102 refactor pagination controls into a separate component * #102 simplified pagination props

Merge branch 'main' into feat/s3-file-browser

ed9be69

Feat/s3 file browser refactor object table (#120)

184fa84

* refactor(ObjectTable): extract File and Folde rows into seperate components * style: format code * style: use camelCase * style: use camelCase

Bohreromir requested changes Sep 17, 2026

View reviewed changes

F3l1x1vo added 8 commits September 18, 2026 08:42

#208 remove TECH_DEBT mentions

5121fcb

#208 implement changes suggested in code review

5db9aaf

#208 use eslint size limits

71d8828

#208 use eslint check-file

acda359

#208 merge db migrations

ec02386

#208 implement changes suggested in code review

17bdaa9

Merge main into feat/s3-file-browser-v1

063f694

#208 fix broken tests and lint from merge

98b4f20

Bohreromir requested changes Sep 18, 2026

View reviewed changes

Comment thread src/lib/server/storage/encryption.ts

Comment thread src/lib/storage/errors.ts Outdated

Comment thread src/lib/storage/tabs.svelte.ts Outdated

Comment thread src/lib/storage/state.svelte.ts Outdated

F3l1x1vo added 2 commits September 21, 2026 07:52

#208 implement changes suggested in code review

3cebf6d

#208 implement changes suggested in code review

895cd8f

Bohreromir requested changes Sep 21, 2026

View reviewed changes

F3l1x1vo added 2 commits September 22, 2026 06:41

#208 implement changes suggested in code review

9a8a09b

#208 remove username fallback for email

0a68c9f

Bohreromir requested changes Sep 22, 2026

View reviewed changes

Comment thread dev/modules/env.sh Outdated

Comment thread dev/modules/summary.sh Outdated

Comment thread dev/modules/postgresql.sh Outdated

Comment thread dev/modules/garage.sh Outdated

Comment thread src/lib/server/db.ts

#208 implement changes suggested in code review

d2ab0c1

#208 implement changes suggested in code review

9e2ec68

#208 fix lint

24b21e6

Bohreromir requested changes Sep 23, 2026

View reviewed changes

F3l1x1vo added 3 commits September 24, 2026 08:20

#208 implement changes suggested in code review

b5a3ed5

#208 add db password for e2e tests

aca3f2f

#208 add conflict resolve to rename

d25554a

Bohreromir requested changes Sep 28, 2026

View reviewed changes

#208 add STORAGE_ENCRYPTION_KEY to .env.example

1468557

Uh oh!

Conversation

dklOrdix commented Jun 18, 2026 • edited by F3l1x1vo Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Bohreromir left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Bohreromir commented Sep 22, 2026 • edited by F3l1x1vo Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Manage Connections

Bucketview

Preview

Uh oh!

F3l1x1vo commented Sep 23, 2026

Uh oh!

Bohreromir commented Sep 23, 2026

Uh oh!

F3l1x1vo commented Sep 23, 2026

Uh oh!

Bohreromir left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Bohreromir Sep 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

dklOrdix commented Jun 18, 2026 •

edited by F3l1x1vo

Loading

Bohreromir commented Sep 22, 2026 •

edited by F3l1x1vo

Loading

Bohreromir Sep 23, 2026 •

edited

Loading