Fix Rollup Job deep-link bug.

elastic · Jul 7, 2020 · cfe0b7a · cfe0b7a
1 parent c3c471d
commit cfe0b7a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_list/job_table/job_table.js b/x-pack/plugins/rollup/public/crud_app/sections/job_list/job_table/job_table.js
@@ -265,7 +265,7 @@ export class JobTable extends Component {
           <EuiLink
             onClick={() => {
               trackUiMetric(METRIC_TYPE.CLICK, UIM_SHOW_DETAILS_CLICK);
-              openDetailPanel(job.id);
+              openDetailPanel(encodeURIComponent(job.id));
             }}
           >
             {value}

diff --git a/x-pack/plugins/rollup/public/crud_app/store/actions/create_job.js b/x-pack/plugins/rollup/public/crud_app/store/actions/create_job.js
@@ -102,7 +102,7 @@ export const createJob = (jobConfig) => async (dispatch) => {
   // here, because it would partially obscure the detail panel.
   getRouter().history.push({
     pathname: `/job_list`,
-    search: `?job=${jobConfig.id}`,
+    search: `?job=${encodeURIComponent(jobConfig.id)}`,
   });
 };