actions/runner
1
0
Fork 0
mirror of https://github.com/actions/runner.git synced 2025-12-10 20:36:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

Clean up file path for error message

Browse Source
This commit is contained in:
Ethan Chiu
2020-06-23 14:19:51 -04:00
parent 368b6254ed
commit 0a6453e241
1 changed files with 12 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
src/Runner.Worker/ActionManifestManager.cs
View File

@@ -34,10 +34,13 @@ namespace GitHub.Runner.Worker
public sealed class ActionManifestManager : RunnerService, IActionManifestManager
{
private TemplateSchema _actionManifestSchema;
private IHostContext _hostContext;
public override void Initialize(IHostContext hostContext)
{
base.Initialize(hostContext);
_hostContext = hostContext;
var assembly = Assembly.GetExecutingAssembly();
var json = default(string);
using (var stream = assembly.GetManifestResourceStream("GitHub.Runner.Worker.action_yaml.json"))
@@ -56,13 +59,19 @@ namespace GitHub.Runner.Worker
{
var templateContext = CreateContext(executionContext);
ActionDefinitionData actionDefinition = new ActionDefinitionData();
// Clean up file name real quick
// Instead of using Regex which can be computationally expensive,
// we can just remove the # of characters from the fileName according to the length of the basePath
string basePath = _hostContext.GetDirectory(WellKnownDirectory.Actions);
var fileName = manifestFile.Remove(0, basePath.Length + 1);
try
{
var token = default(TemplateToken);
// Get the file ID
var fileId = templateContext.GetFileId(manifestFile);
var fileName = templateContext.GetFileName(fileId);
var fileId = templateContext.GetFileId(fileName);
// Add this file to the FileTable in executionContext
executionContext.FileTable.Add(fileName);
@@ -124,7 +133,7 @@ namespace GitHub.Runner.Worker
executionContext.Error(error.Message);
}
throw new ArgumentException($"Fail to load {manifestFile}");
throw new ArgumentException($"Fail to load {fileName}");
}
if (actionDefinition.Execution == null)