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

Ancla Updates #589

Closed
wants to merge 4 commits into from
Closed

Ancla Updates #589

wants to merge 4 commits into from

Conversation

@maurafortino maurafortino self-assigned this Nov 25, 2024
@maurafortino maurafortino marked this pull request as draft November 25, 2024 16:20
Copy link

guardrails bot commented Nov 25, 2024

⚠️ We detected 1 security issue in this pull request:

Vulnerable Libraries (1)
Severity Details
High pkg:golang/github.com/xmidt-org/ancla@v0.3.13-0.20241121184948-3980858531ae upgrade to: > v0.3.13-0.20241121184948-3980858531ae

More info on how to fix Vulnerable Libraries in Go.


👉 Go to the dashboard for detailed results.

📥 Happy? Share your feedback with us.

webhookConfig := in.WebhookConfig
webhookConfig.Logger = in.Logger
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is all being replaced by the anclafx.Provide function:

tr1d1um/main.go

Line 194 in 626bc46

anclafx.Provide(),

ancla: https://github.com/xmidt-org/ancla/blob/feat/webhook-update/anclafx/provide.go

@denopink
Copy link
Contributor

denopink commented Jan 7, 2025

no longer needed

@denopink denopink closed this Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

2 participants