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

Update Trusted-Image-Domains.json #10072

Merged
merged 1 commit into from
Jul 17, 2024
Merged

Update Trusted-Image-Domains.json #10072

merged 1 commit into from
Jul 17, 2024

Conversation

guibranco
Copy link
Contributor

@guibranco guibranco commented Jul 13, 2024

Summary of the changes (in less than 80 characters):

  • Add the app.deepsource.com domain to the trusted image domains list.

Close #9953

Add `app.deepsource.com` domain to trusted image domains list
@guibranco guibranco requested a review from a team as a code owner July 13, 2024 18:49
@erdembayar erdembayar requested a review from lyndaidaii July 16, 2024 22:36
@erdembayar
Copy link
Contributor

@lyndaidaii
Could you please take a look into this?

@lyndaidaii
Copy link
Contributor

Looks good to me. Also reviewed this domain with privacy and security side. looks ok to include it for our allowlist.

@lyndaidaii lyndaidaii merged commit 6708917 into NuGet:main Jul 17, 2024
2 checks passed
@guibranco guibranco deleted the patch-4 branch July 17, 2024 23:10
@lyndaidaii lyndaidaii self-assigned this Jul 30, 2024
@lyndaidaii lyndaidaii added this to the Sprint 2024-07 milestone Jul 30, 2024
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.

[Feature]: Allow readme images from DeepSource
3 participants