iRODS file system setup UX

rspace-os / rspace-web

This the main repository for RSpace: rspace-web. Check https://github.com/rspace-os/rspace-web/blob/main/DevDocs/DeveloperNotes/GettingStarted/GettingStarted.md for setup instructions.

https://github.com/rspace-os/rspace-web/wiki

GNU Affero General Public License v3.0

11 stars 3 forks source link

iRODS file system setup UX #39

Open ll4strw opened 1 month ago

ll4strw commented 1 month ago

Hello RSpace team, I propose the following changes to the page/routines where a sysadmin sets up a remote iRODS file system integration.

The label URL should be substituted by iRODS Host and a tooltip should hint that an iRODS host can be either a hostname or an IP address without protocol.
Set a default iRODS port equal to 1247 if none specified. Currently if no port is specified, any user trying to add an iRODS disk gets an error like Something went wrong: For input string: "".
Currently, this integration supports only the iRODS native authentication method. It would be nice if the auth method defaulted to native if a sysadmin sets up an iRODS integration, just to avoid confusion and to make it more specific to iRODS.
The label Home Directory does not seem to be appropriate in the form to set up a remote iRODS file system by a systemadmin. Instead, I propose iRODS Search Path because that is the root path to search for users collections within iRODS.

mKowalski256 commented 1 month ago

@tilorspace @rs-nicof could you evaluate these suggestions?

Part of them was already implemented by @ll4strw in suggested PR#40. I've started test AWS server for previewing these changes.

rs-nicof commented 1 month ago

@tilorspace @rs-nicof could you evaluate these suggestions?

Part of them was already implemented by @ll4strw in suggested PR#40. I've started test AWS server for previewing these changes.

Hi all, to me I thibnk they are all sensible and good improvements. My doubt is on the bullet point 3 (the native method). I do not fullyt understand what is being asked, can we possibly have an use case?

ll4strw commented 1 month ago

@rs-nicof Bullet point 3 is just a minor aesthetic modification/suggestion. The default authentication method in iRODS is called native, but it is essentially a username/password method. By using the label native in the setup form, we could just make that form more iRODS specific (centric).

ll4strw commented 1 month ago

I just noticed some minor changes that I need to add to #40 to make sure that the custom iRODS labels do not affect the labels for other storage types.

mKowalski256 commented 1 month ago

@tilorspace @rs-nicof I've restarted test instance with latest changes, check: https://irods-ui-changes-4.researchspace.com.

rs-nicof commented 1 month ago

@tilorspace @rs-nicof I've restarted test instance with latest changes, check: https://irods-ui-changes-4.researchspace.com.

@mKowalski256 It looks like sysadmin user does not login for some reason, plus If you connect with the user1 the fielstore is not enabled