Skip to content

Rewritten sftp api#1167

Merged
bajrangCoder merged 10 commits intoAcode-Foundation:mainfrom
bajrangCoder:sftp-ssh-stuff
Feb 5, 2025
Merged

Rewritten sftp api#1167
bajrangCoder merged 10 commits intoAcode-Foundation:mainfrom
bajrangCoder:sftp-ssh-stuff

Conversation

@bajrangCoder
Copy link
Member

@bajrangCoder bajrangCoder commented Jan 26, 2025

  • using native sftp api for sftp fs operation

    • lsDir
    • stat
    • read/write
    • delete(rm)
    • mkdir
    • rename
    • move
    • copy
  • better symlink support

  • better error handling

  • better buffer handling for video etc

  • some performance improvement

  • fix: show info dialog issue in case of non editor tab

  • fix: detect binary files without reading it

  • fix: logger for limiting it's size

  • better visuals for symlinks etc in file browser

  • emoji/unicode support in sftp

  • tweaked breadcrumbs of file browser
    Screenshot_20250127-135908

  • update ssh library

  • handle SFTP related errors

  • [ ] add ssh terminal api for terminal support

- using native sftp api for sftp fs operation
  - [x] lsDir
  - [x] stat
  - [x] read/write
  - [x] delete(rm)
  - [x] mkdir
  - [ ] rename
  - [ ] move
  - [ ] copy

- better symlink support
- better error handling
- better buffer handling for video etc
- some performance improvement
- fix: show info dialog issue because of non editor tab
- [ ] better visuals for symlinks etc in file browser
- [x] emoji/unicode support in sftp
- [ ] add ssh terminal api for terminal support
@bajrangCoder bajrangCoder changed the title Rewritten sftp api and architecture Rewritten sftp api Jan 26, 2025
@feridmoh
Copy link

HI

- [x] rename, move fs api
- [] copy (broken)
- skip listing . and .. in lsdir
- proper exists check in stat
- some cleanups
@bajrangCoder bajrangCoder marked this pull request as ready for review January 27, 2025 15:28
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.

4 participants