[5.2.1] | Fix SqlConnection.FireInfoMessageEventOnUserErrors when set to true throws an exception (#2399)

[DavoudEshtehari]

Ports #2399

[arellegue]

LGTM.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.63%. Comparing base (82be9a6) to head (036bcb0).
Report is 2 commits behind head on release/5.2.

❗ Current head 036bcb0 differs from pull request most recent head c2bca36

Please upload reports for the commit c2bca36 to get more accurate results.

Additional details and impacted files

@@             Coverage Diff              @@
##           release/5.2    #2505   +/-   ##
============================================
  Coverage        72.63%   72.63%           
============================================
  Files              310      310           
  Lines            61875    61875           
============================================
+ Hits             44942    44943    +1     
+ Misses           16933    16932    -1     

Flag	Coverage Δ
addons	92.88% <ø> (ø)
netcore	76.83% <100.00%> (-0.03%)	⬇️
netfx	70.18% <100.00%> (+<0.01%)	⬆️

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

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