iRODS download implementation is commented out

[wilke]

Summary

The iRODS download implementation (~133 lines) is completely commented out, making iRODS a non-functional backend despite having full configuration support.

Location

shock-server/node/util.go:517-649 - commented out implementation
shock-server/conf/conf.go:65-72 - IRodsLocation struct defined

Configuration Struct (defined but unused)

type IRodsLocation struct {
    Zone     string
    User     string
    Password string
    Hostname string
    Port     int
}

Impact

• iRODS is configured as a location type but cannot download files
• The switch statement references it but case is commented: // case "IRods"
• Users configuring iRODS locations will experience silent failures

Recommendation

Either restore/fix the iRODS implementation or remove the configuration support to avoid confusion.

Priority

Medium - non-functional backend with full config support