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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhancement: Trigger project status notifications based on scancode-config.yml values #1208

Open
DennisClark opened this issue May 6, 2024 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@DennisClark
Copy link

See related issue about the scancode-config.yml file: #1197

If a project is created and executed with a scancode-config.yml file, and if that file contains a notification_address, send a notification (email or similar) to that address containing:

  • the text of the "run log" normally made available via SCIO UI status field (e.g. "Success")
  • all of the links normally made available via SCIO UI buttons associated with "Download results as:" so that the recipient can access those links to obtain those result files.
@tdruez
Copy link
Contributor

tdruez commented May 15, 2024

@DennisClark I need your help providing the layout and text content of the email notifications.

@tdruez
Copy link
Contributor

tdruez commented May 30, 2024

@DennisClark I still need your input on the above to make more progress.

@DennisClark
Copy link
Author

DennisClark commented Jun 3, 2024

email subject: Scan results for {project name}

email text:
Project scanned: {project name}
ScanCode.io version: {version}
Scan date: {scan completion timestamp}
Scan status: {status label}
Download log file: {log file link}
Download results as JSON: {JSON link}
Download results as XLXS: {XLXS link}
Download results as SPDX 2.3: {SPDX link}
Download results as CycloneDX 1.4: {CDX 1.4 link}
Download results as CycloneDX 1.5: {CDX 1.5 link}
Download results as CycloneDX 1.6: {CDX 1.6 link}
Download results as Attribution document: {Attribution link}

@DennisClark
Copy link
Author

DennisClark commented Jun 3, 2024

@pombredanne please comment on this from a functionality perspective: do we even need this ? If the results of the command-line scan are written to a specified location anyway, perhaps the next thing to do is to open in ScanCode.io if you want more information. Perhaps an email with the first few elements, down to the log file (perhaps included in the body of the email) would be sufficient.

@tdruez
Copy link
Contributor

tdruez commented Jun 4, 2024

This feature would only make sense for a ScanCode.io server setup as it requires an email service provider to be configured.
Also, if the data is not persistent (ie a command line execution) there are no "Download results ...." possible.

We should promote the, already implemented, usage of Webhook over email as it does not require any extra service to be configured and available. We are lacking documentation though, see #1027

@aboutcode-org aboutcode-org deleted a comment from mjherzog Jun 5, 2024
@pombredanne
Copy link
Contributor

@DennisClark I agree that we may not need this just now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants