Skip to content

Commit 049116f

Browse files
Remove encoded ids from job/dataset error report email
xref galaxyproject#20624 (comment)
1 parent b48e975 commit 049116f

1 file changed

Lines changed: 5 additions & 8 deletions

File tree

lib/galaxy/tools/errors.py

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
This error report was sent from the Galaxy instance hosted on the server
2121
"${host}"
2222
-----------------------------------------------------------------------------
23-
This is in reference to dataset id ${dataset_id} (${dataset_id_encoded}) from history id ${history_id} (${history_id_encoded})
23+
This is in reference to dataset id ${dataset_id_encoded} from history id ${history_id_encoded}
2424
-----------------------------------------------------------------------------
2525
You should be able to view the history containing the related history item (${hda_id_encoded})
2626
@@ -36,7 +36,7 @@
3636
${message}
3737
-----------------------------------------------------------------------------
3838
info url: ${hda_show_params_link}
39-
job id: ${job_id} (${job_id_encoded})
39+
job id: ${job_id_encoded}
4040
tool id: ${job_tool_id}
4141
tool version: ${tool_version}
4242
job pid or drm id: ${job_runner_external_id}
@@ -69,8 +69,8 @@
6969
<h3>Error Localization</h3>
7070
<table style="margin:1em">
7171
<tbody>
72-
<tr><td>Dataset</td><td><a href="${hda_show_params_link}">${dataset_id} (${dataset_id_encoded})</a></td></tr>
73-
<tr style="background-color: #f2f2f2"><td>History</td><td><a href="${history_view_link}">${history_id} (${history_id_encoded})</a></td></tr>
72+
<tr><td>Dataset</td><td><a href="${hda_show_params_link}">${dataset_id_encoded}</a></td></tr>
73+
<tr style="background-color: #f2f2f2"><td>History</td><td><a href="${history_view_link}">${history_id_encoded}</a></td></tr>
7474
<tr><td>Failed Job</td><td>${hid}: ${history_item_name} (${hda_id_encoded})</td></tr>
7575
</tbody>
7676
</table>
@@ -90,7 +90,7 @@
9090
9191
<table style="margin:1em">
9292
<tbody>
93-
<tr><td>Job ID</td><td>${job_id} (${job_id_encoded})</td></tr>
93+
<tr><td>Job ID</td><td>${job_id_encoded}</td></tr>
9494
<tr style="background-color: #f2f2f2"><td>Tool ID</td><td>${job_tool_id}</td></tr>
9595
<tr><td>Tool Version</td><td>${tool_version}</td></tr>
9696
<tr style="background-color: #f2f2f2"><td>Job PID or DRM id</td><td>${job_runner_external_id}</td></tr>
@@ -195,16 +195,13 @@ def create_report(self, user, email="", message="", redact_user_details_in_bugre
195195
report_variables = dict(
196196
host=host,
197197
dataset_id_encoded=self.app.security.encode_id(hda.dataset_id),
198-
dataset_id=hda.dataset_id,
199198
history_id_encoded=history_id_encoded,
200-
history_id=hda.history_id,
201199
hda_id_encoded=hda_id_encoded,
202200
hid=hda.hid,
203201
history_item_name=hda.get_display_name(),
204202
history_view_link=history_view_link,
205203
hda_show_params_link=hda_show_params_link,
206204
job_id_encoded=self.app.security.encode_id(job.id),
207-
job_id=job.id,
208205
tool_version=job.tool_version,
209206
job_tool_id=job.tool_id,
210207
job_tool_version=hda.tool_version,

0 commit comments

Comments
 (0)