-
Notifications
You must be signed in to change notification settings - Fork 5
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
}, | ||
&cli.StringFlag{ | ||
Name: "url", | ||
Value: "localhost:10054", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reserve a port
@@ -136,7 +136,7 @@ func SharingWithConfig(cfg *config.Config) []cli.Flag { | |||
}, | |||
&cli.StringFlag{ | |||
Name: "public-driver", | |||
Value: "memory", | |||
Value: "json", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the json driver has a hard dependency on its db file. This might fail a default ocis run.
|
|
Here is an overview of what got changed by this pull request: Complexity increasing per file
==============================
- pkg/flagset/storagepubliclink.go 1
- pkg/command/storagepubliclink.go 8
Clones added
============
- pkg/command/frontend.go 1
- pkg/command/storageocdata.go 2
- pkg/flagset/storagepubliclink.go 18
- pkg/command/storageroot.go 2
- pkg/command/storagepubliclink.go 10
- pkg/command/storageoc.go 2
Clones removed
==============
+ pkg/command/gateway.go -1
See the complete overview on Codacy |
seems we still have hard-coded localhost:10054 ? can we sort this out and make it configurable ? |
to solve the CI, please rebase on top of origin/master from the owncloud/ocis-reva repo. |
closing in favor of #232 |
Provide
reva-storage-public-link
command and its default UNIX configuration.