More rows.Close() and rows.Err()

[S7evinK]

Looks like we missed some rows.Close()

Even though rows.Err() is mostly not necessary, we should be more consistent in the DB layer.

[codecov]

Codecov Report

Attention: 26 lines in your changes are missing coverage. Please review.

Comparison is base (ee73a90) 65.51% compared to head (434441d) 65.49%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3262      +/-   ##
==========================================
- Coverage   65.51%   65.49%   -0.02%     
==========================================
  Files         507      507              
  Lines       57217    57263      +46     
==========================================
+ Hits        37484    37507      +23     
- Misses      15892    15909      +17     
- Partials     3841     3847       +6     

Flag	Coverage Δ
unittests	49.48% <72.91%> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
federationapi/storage/postgres/queue_json_table.go	70.00% <100.00%> (ø)
...erationapi/storage/postgres/relay_servers_table.go	74.46% <100.00%> (ø)
...ederationapi/storage/sqlite3/joined_hosts_table.go	64.13% <100.00%> (ø)
federationapi/storage/sqlite3/queue_json_table.go	65.00% <100.00%> (ø)
...derationapi/storage/sqlite3/relay_servers_table.go	73.21% <100.00%> (ø)
...elayapi/storage/postgres/relay_queue_json_table.go	70.00% <100.00%> (ø)
relayapi/storage/sqlite3/relay_queue_json_table.go	65.00% <100.00%> (ø)
roomserver/storage/postgres/membership_table.go	79.70% <100.00%> (ø)
roomserver/storage/postgres/rooms_table.go	85.00% <100.00%> (ø)
...oomserver/storage/postgres/user_room_keys_table.go	82.71% <100.00%> (+0.43%)	⬆️
... and 40 more

... and 4 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.