fix(diagnostics): treat _debug=128 as debug on
Build / Build-and-ng-test (pull_request) Successful in 5m24s
Lighthouse Checks / lighthouse (pull_request) Successful in 20m58s
Build / Build-and-test-development (pull_request) Successful in 25m23s

The adapter sends _debug=128 rather than 131 on the Viya web (JES) path
when runAsTask is enabled - which is the path the frontend uses there -
so the extra mpeinit logging, including the browser_info /
browser_url_vars dump, never fired on those estates. Accept 128
alongside 131 (and the existing 2477 / fields,log,trace values).

Verified on a Viya estate: the same service with _debug=128 logs
'no work.browser_info on this request' with the fix and logs nothing
without it.
This commit is contained in:
dc committed 2026-09-25 08:22:14 +00:00
1 parent 293636a5f8
commit 8de9978213
1 file changed
+5
+5
View File
@@ -115,7 +115,12 @@ run;
,msg=%str(Problem during compilation or with STP precode (&syswarningtext))
)
/* `_debug` is 131 by default, but the adapter sends 128 on the Viya WEB JES
* path when runAsTask is enabled (see WebJobExecutor.getRequestParams), and
* 128 also arrives when a `log` debug value is requested. All of them mean
* "debug on" - so all of them enable the extra logging below. */
%if "&_debug"="2477" or "&_debug"="fields,log,trace" or "&_debug"="131"
or "&_debug"="128"
%then %do;
%let sasjs_mdebug=1;
%end;