codeforIATI / iati-data-bugtracker

🐛 A public log of issues with IATI data and metadata
https://bugtracker.codeforiati.org
3 stars 0 forks source link

Consider sanity limitations for all strings #3

Closed notshi closed 5 years ago

notshi commented 5 years ago

For example, according to the standard:

transaction@ref An internal reference linking this transaction back to the publisher’s financial management system. This value must be of type xsd:string.

However, there are no character limitations which means the following is valid data. Screenshot-20190409002853-1899x847 Source: Aidstream

And these are valid iati-identifier. Screenshot-20190409004156-680x572 Source: Registry

We found this because Postgres complains about text that is more than 8kb when creating indexes.

andylolz commented 5 years ago

This sounds like a proposed schema change. As such, it would be good to raise it on IATI Discuss, probably in "Modifications, Additions, Improvements": https://discuss.iatistandard.org/c/standard-management/modifications-additions-improvements