whitespace-padding should be stripped from item barcodes
Description
Leading and trailing whitespace should be stripped from item-record barcodes before they are POSTed. Supplying an item barcode with leading or trailing whitespace causes the circulation/check-out-by-barcode API endpoint to fail with a 500 status code and the response,
Failed to contact storage module: java.io.IOException: Connection reset by peer
Entering a user barcode with leading or trailing whitespace causes no ill effect.
Tested leading and trailing barcode spaces in check in, check out and requests. All worked great. Thanks !
Zak BurkeApril 8, 2019 at 4:35 PM
, I have PRs up to resolve this issue by trimming whitespace on the UI side, but I wonder if it might be better to handle this server side in addition/instead. What do you think?
Leading and trailing whitespace should be stripped from item-record barcodes before they are POSTed. Supplying an item barcode with leading or trailing whitespace causes the
circulation/check-out-by-barcode
API endpoint to fail with a 500 status code and the response,Entering a user barcode with leading or trailing whitespace causes no ill effect.