fix(server): Fix the issue where GetArtifact didn't look for input artifacts

[dinever]

Fixes #5704

[codecov]

Codecov Report

Merging #5705 (28b83df) into master (91c08cd) will increase coverage by 0.05%.
The diff coverage is 55.55%.

❗ Current head 28b83df differs from pull request most recent head 78a360e. Consider uploading reports for the commit 78a360e to get more accurate results

@@            Coverage Diff             @@
##           master    #5705      +/-   ##
==========================================
+ Coverage   46.91%   46.96%   +0.05%     
==========================================
  Files         244      244              
  Lines       15209    15228      +19     
==========================================
+ Hits         7135     7152      +17     
  Misses       7169     7169              
- Partials      905      907       +2     

Impacted Files	Coverage Δ
workflow/executor/executor.go	16.18% <0.00%> (-0.06%)	⬇️
workflow/executor/resource.go	22.80% <0.00%> (ø)
workflow/cron/controller.go	0.75% <33.33%> (+0.75%)	⬆️
workflow/controller/operator.go	71.12% <50.00%> (+0.06%)	⬆️
server/artifacts/artifact_server.go	59.32% <84.61%> (+2.04%)	⬆️
server/cronworkflow/cron_workflow_server.go	65.07% <100.00%> (+3.01%)	⬆️
cmd/argo/commands/get.go	56.95% <0.00%> (+0.64%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 91c08cd...78a360e. Read the comment docs.

[dinever]

Thanks @alexec!

I've pushed the latest commit which adds two new endpoints to support input artifacts:

/input-artifacts/:namespace/:workflowName/:nodeId/:artifactName
/input-artifacts-by-uid/:uid/:nodeId/:artifactName

The original URL for output artifacts was not changed. The handler functions were renamed to GetOutputArtifacts... tho for clarification.

Please take a look when available. Thanks!