HUE-8564 [useradmin] Fix last activity update for notebook/api/check_status

Review Request #13387 - Created Sept. 24, 2018 and submitted

Ying Chen
hue
master
HUE-8564
hue
enricoberti, jgauthier, johan, ranade, romain, weixia
commit d732615daff7a5db93981d2d731c9ff615653cf7 (HEAD -> master)
Author: Ying Chen <yingchen@cloudera.com>
Date:   Mon Sep 24 17:53:30 2018 -0700

    HUE-8564 [useradmin] Fix last activity update for notebook/api/check_status

:100644 100644 073d839a7e... ea14601a16... M    apps/useradmin/src/useradmin/middleware.py


  • 0
  • 0
  • 0
  • 1
  • 1
Description From Last Updated
  1. 
      
  2. apps/useradmin/src/useradmin/middleware.py (Diff revision 1)
     
     
     
     
     
    nit:
    How about being 100% consistent and go on new line for each item and also strip all of them, e.g.:
    
        if request.POST.get('format') == 'json') \
    and not request.path.strip('/') in (
      'notebook/api/check_status',
      'jobbrowser/api/jobs',
      'jobbrowser/jobs',
      'desktop/debug/is_idle'
    ):
    1. request.POST.get('format') == 'json' can't be applied to most of the Ajax calls like '/notebook/api/get_logs', '/notebook/api/fetch_result_data', or '/notebook/api/fetch_result_size'
  3. 
      
Review request changed

Status: Closed (submitted)

Loading...