search

unioslo / mreg

GNU General Public License v3.0
7 stars 13 forks source link

Log unexpected input to DjangoJSONModelEncoder. #500

Closed terjekv closed 1 year ago

terjekv commented 1 year ago

https://github.com/unioslo/mreg/commit/014e25b6030a05adf9622694539b92a38f71a2b1 removed the calls to super().default(o) and thus broke the case where DjangoJSONModelEncoder was fed a datetime.date object.

This was resolved in https://github.com/unioslo/mreg/pull/496, but it would be good to know if there is any other unexpected input coming into the encoder.

coveralls commented 1 year ago

Coverage Status

Coverage: 98.835% (-0.03%) from 98.865% when pulling ad6298a5beff72f771281deb58fa6d0749b441ce on terjekv:jsondecoder_weirdness into 7c6f4900593242897fc239ca9da0c14b398a2e88 on unioslo:master.