Skip to content

Conversation

greglittlefield-wf
Copy link
Collaborator

@greglittlefield-wf greglittlefield-wf commented Jan 29, 2021

There was a hyphen before a comment that shouldn't have been there, causing Aviary to error

@aviary3-wk
Copy link

aviary3-wk commented Jan 29, 2021

Security Insights

The items listed below may not capture all security relevant changes. Before providing a security review, be sure to review the entire PR for security impact.

PR and Jira ticket analysis indicate this PR has security relevant changes.

Error scanning for security relevant changes
Error scanning for potential secrets
Error scanning for cfn validation issues

Action Items

  • Request infosec review via Slack: #support-infosec
  • Verify aviary.yaml coverage of security relevant code

Questions or Comments? Reach out on Slack: #support-infosec.

@semveraudit-wf
Copy link

semveraudit-wf commented Jan 29, 2021

Public API Changes

No changes to the public API found for commit aee9a57

Showing results for aee9a57

Powered by semver-audit-service. Please report any problems by filing an issue.
Reported by the dart semver audit client 2.2.0
Browse public API.

Last edited UTC Jan 29 at 18:24:06

Copy link

@jeffreymelvin-wf jeffreymelvin-wf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 security

@jeffreymelvin-wf
Copy link

aviary override required

@jeffreymelvin-wf
Copy link

+1 security

Copy link
Collaborator

@aaronlademann-wf aaronlademann-wf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

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

Successfully merging this pull request may close these issues.

7 participants