The Wayback Machine - https://web.archive.org/web/20240531124909/https://github.com/oasis-tcs/sarif-spec/issues/92
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add stdin/stdout/stderr on invocation #92

Closed
ghost opened this issue Feb 12, 2018 · 1 comment
Closed

Add stdin/stdout/stderr on invocation #92

ghost opened this issue Feb 12, 2018 · 1 comment

Comments

@ghost
Copy link

ghost commented Feb 12, 2018

runs:
[
  {
    "invocation": {
      "stdin": "...",
      "stdout": "...",
      "stderr": "...",
      ...
    }
  }
]
@ghost ghost added the CSD.1 Will be fixed in CSD.1. label Feb 12, 2018
@ghost ghost self-assigned this Feb 12, 2018
ghost pushed a commit that referenced this issue Feb 12, 2018
@michaelcfanning
Copy link
Contributor

The current change draft resulted in some suggestions that will reset this change draft. At a high level, it was noted that this content could be copious (the question was posed whether we should limit the size of the string value). The primary suggestion was to make the property value a physical location. The URI for this could be a reference into content embedded in the files table, as usual. The physical location could also, presumably, refer to a region in a separate log file.

This suggestion minimizes complexity in the format but we lose the possibility of seeing the console output in clear text in the log file (which the TC seems comfortable with).

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant