View Issue Details

IDProjectCategoryView StatusLast Update
0005715Talerobsolete componentpublic2019-12-20 18:12
ReporterFlorian Dold Assigned ToMarcello Stanisci Resolution deadline
ImplementerReviewer 
Release signed byDeployed by 
VerifierCommunicator 
Data protection authority notification requiredNo  
CSIRT notification requiredNo  
Estimated effort (hours)Actual effort (hours)0 
Fast-trackNo
Work packagenone
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product Versiongit (master) 
Target Version0.6Fixed in Version0.6 
Summary0005715: stop using validictory
DescriptionWe are using a library for input validation called "validictory" [1]. This library is ugly and also deprecated/unsupported. The way we're using this library is a complete mess [2]. Changing one parameter becomes and adventure of chasing down odd schema definitions. While there are much better python libraries available [3], why don't we just use Django's built-in form validation? That also works for GET parameters.

[1] https://github.com/jamesturk/validictory
[2] https://git.taler.net/bank.git/tree/talerbank/app/schemas.py?id=801d7032206cd89477bc201eccb5da47a5b975f2#n319
[3] https://valedictory.readthedocs.io/en/latest/ (sounds similar, but is completely different!)
[4] https://pypi.org/project/jsonschema/ also kind of ugly, but at least based on a standard
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Marcello Stanisci

2019-05-31 18:31

reporter   ~0014476

Purging Validictory from commit: c43180054aa94409.

Next change should revise the way the exception middleware constructs error responses, but that's not strictly related to this one, therefore I am going to close here.

Issue History

Date ModifiedUsernameFieldChange
2019-05-08 05:07Florian DoldNew Issue
2019-05-08 05:07Florian DoldStatusnew => assigned
2019-05-08 05:07Florian DoldAssigned To => Marcello Stanisci
2019-05-31 18:31Marcello StanisciNote Added: 0014476
2019-05-31 18:31Marcello StanisciStatusassigned => informing
2019-05-31 18:31Marcello StanisciResolutionopen => fixed
2019-06-01 08:57Christian GrothoffProduct Version => git (master)
2019-06-01 08:57Christian GrothoffFixed in Version => 0.6
2019-06-01 08:57Christian GrothoffTarget Version => 0.6
2019-12-20 18:12Christian GrothoffStatusinforming => closed
2022-08-23 18:26Christian GrothoffCategorybank (demonstrator) => py bank (demonstrator, obsolete)
2023-12-03 00:23Christian GrothoffCategorypy bank (demonstrator, obsolete) => obsolete componet
2023-12-11 19:08Florian DoldCategoryobsolete componet => obsolete component