google / exposure-notifications-verification-server

Verification component for COVID-19 Exposure Notifications.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Batch upload may report incorrect lines

whaught opened this issue · comments

The line numbering can get off if the file contains empty lines (which is valid)
The code currently assumes batchSize - n sequentially are the lines. Instead we should build a map of line# alongside the request before passing it to ajax, then properly throw it in the error/success table

Unrelated to this bug: while you're here consider skipping lines with the uuid_already_exists code?

/assign