Add limits for Analytics events resending

[powerful23]

Issue #, if available:

Description of changes:
To avoid infinite retries for Analytics resending events.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[codecov-io]

Codecov Report

Merging #590 into master will decrease coverage by 0.03%.
The diff coverage is 86.36%.

@@            Coverage Diff             @@
##           master     #590      +/-   ##
==========================================
- Coverage   89.15%   89.11%   -0.04%     
==========================================
  Files          69       69              
  Lines        3219     3226       +7     
  Branches      611      613       +2     
==========================================
+ Hits         2870     2875       +5     
- Misses        331      333       +2     
  Partials       18       18

Impacted Files	Coverage Δ
packages/aws-amplify/src/Analytics/Analytics.ts	88.23% <86.36%> (-1.24%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e98ad53...f86f32c. Read the comment docs.

[mlabieniec]

Should these values perhaps be configurable via the pluggable / config since different situations / systems may require different values.

[Jun711]

This is to fix this potential bug, right?

When I had window.LOG_LEVEL='DEBUG' on but not using Analytics, I noticed this message is fired continuously.

{"[DEBUG] 23:50.499 AWSAnalyticsProvider - record event failed. ": Error: Missing region in config
at Request.VALIDATE_REGION (webpack-internal:///../../../../aws-…}

amplify version 0.2.14

[powerful23]

@Jun711 Yes. By the way, if you don't want to use Analytics, you can turn it off manually. You can check this doc

[Jun711]

Thanks @powerful23
I don't use awsmobile though. it would be great if I know which line to add in. Thanks.

[powerful23]

@Jun711 https://aws.github.io/aws-amplify/media/analytics_guide#manual-setup

[Jun711]

thanks @powerful23 . after your pull request, I don't have to set disabled to false to disable Analytics?
(resending events retry issue)

[powerful23]

@Jun711 yes

[github-actions]

This pull request has been automatically locked since there hasn't been any recent activity after it was closed. Please open a new issue for related bugs.

Looking for a help forum? We recommend joining the Amplify Community Discord server *-help channels or Discussions for those types of questions.