[ACCUMULO-4762] Synchronous JS calls are deprecated

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Not A Problem
• Affects Version/s: 2.0.0
• Component/s: monitor
• Assignee: Benjamin F
• Reporter: Michael Miller
• Created: 12/Dec/17 19:30
• Updated: 16/Apr/19 19:21
• Resolved: 16/Apr/19 19:21

Description

All of the javascript in the Monitor sets the async flag to false before loading some values in the monitor. Once the values are loaded, we then set the flag to true.

$.ajaxSetup({
    async: false
  });
//... load something
$.ajaxSetup({
    async: true
  });

This currently gives a warning in Chrome since this behavior is deprecated:

[Deprecation] Synchronous XMLHttpRequest on the main thread is deprecated because of its detrimental effects to the end user's experience. For more help, check https://xhr.spec.whatwg.org/.

This may not be a problem now but upon further investigation this could be a problem in the future. According to the XMLHttpRequest standard:
"...Developers must not pass false for the async argument when current global object is a Window object." I look at this as using the Asynchronous JS in a way it was not intended to be used and we could probably do this better.

Fixing this would require some redesign of the Monitor but it would be better to do this now before releasing 2.0.