CPLAT-7607 [Dart 1] Improve handling of “repeat” errors thrown from components wrapped by an ErrorBoundary

[aaronlademann-wf]

• Dart 1 port of CPLAT-7606 Improve handling of “repeat” errors thrown from components wrapped by an ErrorBoundary #350

See #350 for full description

@greglittlefield-wf @kealjones-wk @joebingham-wk @sydneyjodon-wk

[aviary2-wf]

Security Insights

The items listed below may not capture all security relevant changes. Before providing a security review, be sure to review the entire PR for security impact.

(3) Security relevant changes were detected

Watched keyword innerHtml in lib/src/component/error_boundary.dart line(s) ['325'] added

Watched keyword InnerHtml in lib/src/component/error_boundary.dart line(s) ['349'] added

Watched keyword dangerouslySetInnerHTML in lib/src/component/error_boundary.dart line(s) ['350'] added

Action Items

• Obtain a security review; reviewer should pay special attention to insights listed above
• Verify aviary.yaml coverage of security relevant code

---

Questions or Comments? Reach out on Slack: #support-infosec.

[codecov-io]

Codecov Report

Merging #352 into master_dart1 will decrease coverage by 0.06%.
The diff coverage is 81.25%.

@@               Coverage Diff               @@
##           master_dart1    #352      +/-   ##
===============================================
- Coverage         90.16%   90.1%   -0.05%     
===============================================
  Files                36      36              
  Lines              1828    1868      +40     
===============================================
+ Hits               1648    1683      +35     
- Misses              180     185       +5

[maxwellpeterson-wf]

+1 security

• Our CSP protects us from any potential XSS made possible by this change.

[greglittlefield-wf]

+10

[maxwellpeterson-wf]

+1 security refresh

[greglittlefield-wf]

@Workiva/release-management-p