Skip to content
This repository has been archived by the owner on Jan 18, 2021. It is now read-only.

Public Links #133

Closed
wants to merge 20 commits into from
Closed

Public Links #133

wants to merge 20 commits into from

Conversation

refs
Copy link
Member

@refs refs commented Apr 14, 2020

Provide reva-storage-public-link command and its default UNIX configuration.

@update-docs
Copy link

update-docs bot commented Apr 14, 2020

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.

pkg/command/gateway.go Outdated Show resolved Hide resolved
},
&cli.StringFlag{
Name: "url",
Value: "localhost:10054",
Copy link
Member Author

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",
Copy link
Member Author

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.

@refs refs marked this pull request as ready for review April 28, 2020 21:37
@refs refs requested a review from butonic April 28, 2020 21:45
@refs refs self-assigned this Apr 28, 2020
@PVince81
Copy link
Contributor

  • make link grant path configurable

@refs refs changed the title Public Storage Public Links May 27, 2020
@refs
Copy link
Member Author

refs commented May 28, 2020

* [ ]  make link grant path configurable

we got rid of this

@ownclouders
Copy link

Codacy 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

@PVince81
Copy link
Contributor

seems we still have hard-coded localhost:10054 ? can we sort this out and make it configurable ?

@PVince81
Copy link
Contributor

to solve the CI, please rebase on top of origin/master from the owncloud/ocis-reva repo.
it should contain an adjustment in the Drone config related to mimetype setup.

@PVince81
Copy link
Contributor

closing in favor of #232

@PVince81 PVince81 closed this May 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants