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

Don't use azureedge.net CDN #8846

Merged
merged 1 commit into from
Apr 11, 2024
Merged

Don't use azureedge.net CDN #8846

merged 1 commit into from
Apr 11, 2024

Conversation

directhex
Copy link
Contributor

No description provided.

@directhex directhex requested a review from grendello as a code owner March 28, 2024 13:21
@jonpryor
Copy link
Member

Rationale? Is it going away? Were we not supposed to use it in the first place? Other?

@directhex
Copy link
Contributor Author

@jonpryor security compliance requires we have DDoS protection for all CDNs. For various "whoever touched it last is on the hook" reasons, it's on my plate to deal with. And DDoS protection is not possible for anything on an azureedge.net domain (as this domain is tied to a legacy CDN product without DDoS protection capability).

The bare blob storage still works, or if a CDN is desired, someone on a more appropriate team can set one up on the new Azure Front Door product which supports the required compliance needs (e.g. download.mono-project.com switched to the new system a week ago)

@directhex
Copy link
Contributor Author

@grendello is there a timeline on getting this merged?

@grendello
Copy link
Contributor

@directhex should be fine to merge it now IMO

@directhex directhex merged commit a58b21d into main Apr 11, 2024
46 of 48 checks passed
@directhex directhex deleted the azureedgenet branch April 11, 2024 15:12
@directhex
Copy link
Contributor Author

Are there servicing branches this needs to be backported to?

@grendello
Copy link
Contributor

@jonathanpeppers which branches do we need to backport this?

@jonathanpeppers
Copy link
Member

We can probably just backport to .NET 8 for now. 👍

jonathanpeppers pushed a commit that referenced this pull request Apr 11, 2024
grendello added a commit that referenced this pull request Apr 12, 2024
* main:
  Don't use azureedge.net CDN (#8846)
@directhex
Copy link
Contributor Author

How do we feel about scream-testing shutdown of the old CDN soon?

@directhex
Copy link
Contributor Author

Old CDN is turned off, so everyone can get some screaming done

grendello added a commit that referenced this pull request Apr 23, 2024
* main:
  Bump to xamarin/xamarin-android-binutils/L_18.1.4-8.0.0@758d2e7 (#8885)
  [Mono.Android] Bind API-VanillaIceCream Beta 1 (#8891)
  [AndroidToolTask] Log tool output as error  (#8861)
  [Xamarin.Android.Build.Tasks] Remove "Xamarin" from messages (#8884)
  [Mono.Android] Bind API-VanillaIceCream Developer Preview 2 (#8741)
  Bump to dotnet/installer@22ffa42d6c 9.0.100-preview.4.24221.5 (#8887)
  Bump external/xamarin-android-tools from `37d79c9` to `05f9a90` (#8869)
  Bump external/Java.Interop from `e1c7832` to `06214ff` (#8878)
  Bump to dotnet/installer@7380c301c1 9.0.100-preview.4.24215.2 (#8874)
  [Mono.Android] Commit baseline PublicAPI files for API-35 (#8840)
  Add a unit test to check environment processing (#8856)
  Don't use azureedge.net CDN (#8846)
  Bump to dotnet/installer@0bfd2dd757 9.0.100-preview.4.24208.2 (#8862)
  [ci] Update dependabot ignore list (#8864)
  Bump external/Java.Interop from `651de42` to `e1c7832` (#8836)
  Bumps LLVM to v18.1.3 and XA utils version to 8.0.0 (#8852)
grendello added a commit that referenced this pull request Apr 25, 2024
* main: (26 commits)
  [Mono.Android] fix potential leak of RunnableImplementors (#8900)
  [build] Bump $(AndroidNetPreviousVersion) to 34.0.95 (#8898)
  [docs] Reorganize public Documentation (#8889)
  Bump external/Java.Interop from `06214ff` to `6cfa78c` (#8879)
  Localized file check-in by OneLocBuild Task (#8894)
  $(AndroidPackVersionSuffix)=preview.5; net9 is 34.99.0.preview.5 (#8896)
  Bump to xamarin/monodroid@a5742221b3 (#8893)
  Remove MonoArchive_BaseUri from Configurables.cs (#8890)
  Bump to xamarin/xamarin-android-binutils/L_18.1.4-8.0.0@758d2e7 (#8885)
  [Mono.Android] Bind API-VanillaIceCream Beta 1 (#8891)
  [AndroidToolTask] Log tool output as error  (#8861)
  [Xamarin.Android.Build.Tasks] Remove "Xamarin" from messages (#8884)
  [Mono.Android] Bind API-VanillaIceCream Developer Preview 2 (#8741)
  Bump to dotnet/installer@22ffa42d6c 9.0.100-preview.4.24221.5 (#8887)
  Bump external/xamarin-android-tools from `37d79c9` to `05f9a90` (#8869)
  Bump external/Java.Interop from `e1c7832` to `06214ff` (#8878)
  Bump to dotnet/installer@7380c301c1 9.0.100-preview.4.24215.2 (#8874)
  [Mono.Android] Commit baseline PublicAPI files for API-35 (#8840)
  Add a unit test to check environment processing (#8856)
  Don't use azureedge.net CDN (#8846)
  ...
@github-actions github-actions bot locked and limited conversation to collaborators May 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants