Comments (5)
Updated release notes, thanks for noticing. See #1349
I think it should work if you return an existing virtual folder by name.
from sftpgo.
Hi, thank you for the quick reply. I didn't knew about #1349. I was hopping to still be able to create virtual folders on the fly when returning user configuration. It's a bit annoying but I will adapt my approach for my external auth hook.
I think it should work if you return an existing virtual folder by name.
As I was explaining above, it does work for the Web UI but not for SSH (and FTP, WebDAV) for those you need the virtual folder id. This behavior should be consistent.
Error with error creating root directory "/virtual_path" for user "test": mkdir : no such file or directory when listing the virtual folder via SSH, if a virtual folder with the same name exist
One approach could be to change the user/group create/update API virtual_folders
to take only virtual folders virtual_path
, quota_size
, quota_files
, folder_id
instead of full virtual folders configuration since setting other fields will/should not have any impact.
Should we continue this in another issue or reopen this one ?
from sftpgo.
I did a quick test to confirm, returning an existing folder name is enough, the id is not required.
The OpenAPI document should be updated in several places to clarify the fields required for adding/updating and the fields returned in GETs.
If you want to help us and feel comfortable with our CLA feel free to send a PR. The PR should be complete and working to be accepted, we have currently no time for reviewing and iterating or pushing back on PRs (this is often more work than doing the change ourself). Thanks for understanding and for using SFTPGo
from sftpgo.
We also hit this barrier. It seems one way communication needs to be changed as 2 ways communication. So currently we are in between "to create a PR for changing the implementation to one way communication as before" or "implementing this behaviour as is". I am open to create a PR for changing the webhook but I want to be sure before spending time whether it fits with your architectural vision or not.
from sftpgo.
We also hit this barrier. It seems one way communication needs to be changed as 2 ways communication. So currently we are in between "to create a PR for changing the implementation to one way communication as before" or "implementing this behaviour as is". I am open to create a PR for changing the webhook but I want to be sure before spending time whether it fits with your architectural vision or not.
We want to have a separation between users, folders and groups. Historically the hook automatically created folders because folders were originally included with users, but this design had several limitations. Now we also have groups and we don't want the hook to automatically create groups as well, we should do that if we create folders.
from sftpgo.
Related Issues (20)
- [Bug]: Permission Denied While Uploading Files To GCS Bucket
- [Bug]: the token with id \"..\" is not valid for the ip address HOT 2
- two 2FA page does not load when served via cloud flare
- [Bug]: JSch clients can't connect via publickey after upgrading from v2.5.6 to v2.6.2 HOT 10
- Login As Button | Use API to Login As User HOT 1
- [Feature] Keep me logged in (WebUI)
- Decrease msg size to match pkg/sftp client maxMsgSize constant HOT 3
- [Bug]: error creating host key "id_rsa" HOT 22
- Custom Action Hook not working[Bug]:
- Event Manager action of "Create or Update" for "Identity Provider account check" deletes SSH keys from user's profile [Bug]:
- Backstage multilingual support?
- drakkan/sftpgo French translation HOT 2
- Support for Minute-Based Scheduling in Rule Events
- Improve Errors reported by the Web UI when a file upload fails
- Add "shares-disabled" option on a per Virtual Folder basis
- Admins should have access to the WebClient and their user profile
- [Bug]: unable to edit virtual folders beginning with slashes in name HOT 3
- Allow users to create API keys in WebClient UI
- Multifactor Authentication on shares
- [Bug]: Creating an user with non existing virtual folder errors HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from sftpgo.