mirror of
https://github.com/ciur/papermerge.git
synced 2026-04-25 12:05:58 +03:00
[GH-ISSUE #167] Digest validation #130
Labels
No labels
2.1
3.0
3.0.1
3.0.2
3.0.3
3.0.3
3.1
3.2
3.2
3.3
3.5
3.x
Fixed. Waiting for feedback.
Fixed. Waiting for feedback.
UX
Version 2.1 - alpha
XSS
announcement
beta
blocker
bug
cannot reproduce
confirmed
confirmed
critical
demo
dependencies
deployment
detchnical debt
discussion
docker
documentation
donations
duplicate
enhancement
feature request
frontend
fundraising
good first issue
good issue
help wanted
high
implemented
important
improvement
incomplete
invalid
investigation
kubernetes
low
low impact
medium
medium
medium impact
migration from 2.0
migration from 2.1
missing-language
missing-ocr-language
no-activity
note
ocr
outofscope
packaging
performance
popular request
pull-request
pypi
question
raspberry pi
roadmap
search
security
setup
status
task
technical debt
updates
user xp
version 1.4.0 - demo
will be implemented
will not be implemented
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/papermerge#130
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @francescocarzaniga on GitHub (Oct 14, 2020).
Original GitHub issue: https://github.com/ciur/papermerge/issues/167
Originally assigned to: @ciur on GitHub.
I think that having a digest+user validation to prevent duplicate file upload is a good thing. When I have thousands or more documents I would like to avoid importing duplicates.
I understand this makes testing more difficult, but disabling validation in production because they are an inconvenience does not sound too good. There could at least be a setting for it, possibly per-user.
@ciur commented on GitHub (Oct 15, 2020):
@francescocarzaniga, my plans is to make papermerge easy to extend. Your ticket sounds to me like a perfect candidate for plugin/app/add-on for Papermerge.
I am investigating different ways "how to make papermerge extensible". When I will find a reasonable approach, I will document and you would be able to add an extension with "digest validation".
Without a good plugin/app system, we can infinitely debate the pros and cons of digest validation.
@ciur commented on GitHub (Oct 20, 2020):
@francescocarzaniga, the master branch now contains papermerge.core.models.AbstractDocument model - you can use it to write your own (django based apps) plugins to add whatever additional validations you wish.
See pagelimit app as example how to add additional document validations in external apps.
This is still experimental API, but it is already usable. limitpages app is used in demo to limit number of pages for uploaded docs.
You want to use get_absfilepath function to get access to the document's file.