Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Share some files with others #446

Closed
eikek opened this issue Nov 15, 2020 · 2 comments · Fixed by #1133
Closed

Share some files with others #446

eikek opened this issue Nov 15, 2020 · 2 comments · Fixed by #1133

Comments

@eikek
Copy link
Owner

eikek commented Nov 15, 2020

Just an idea – It would be nice to allow to share certain files with some one outside of docspell, for example with your tax advisor. Docspell could generate unique URLs that can be shared that would show only a selected subset of items.

@totti4ever
Copy link
Contributor

totti4ever commented Nov 16, 2020

Most probably it would that way anyway, but it would help a lot if the path for shared file was unique, i.e. docspell.tld/shared or /s or whatsoever

Why? That way one could whitelist only those urls for puclic access in a reverse proxy like traefik

And the shares should be listable and removable (and maybe show the view count)

@eikek
Copy link
Owner Author

eikek commented Nov 16, 2020

Yes, this will be on a different path, because it will be a different page. Very similar to the public source page that exists, like /app/upload/5JEYQa3SfQ…. I also thought about protecting it with a password …. essentially duplicating the functionality from sharry ;-). But here it would be possible to allow searching in only a subset of items.

@eikek eikek added this to the Docspell 0.28.0 milestone Sep 28, 2021
@mergify mergify bot closed this as completed in #1133 Oct 24, 2021
eikek added a commit that referenced this issue Oct 25, 2021
Currently the query implementation cannot combine multiple/nested
fulltext searches within a query. It doesn't seem useful to have
shares based on fulltext searches, so it is disabled for now.

Issue: #446
eikek added a commit that referenced this issue Oct 25, 2021
Currently the query implementation cannot combine multiple/nested
fulltext searches within a query. It doesn't seem useful to have
shares based on fulltext searches, so it is disabled for now.

Issue: #446
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants