Skip to content

Conversation

@rshade
Copy link

@rshade rshade commented May 12, 2025

This pull request introduces support for enabling HTTPS on Azure CDN custom domains in the Pulumi Azure Native provider. It includes updates to the provider's codebase, new SDK resources, and an example demonstrating the feature.

New Feature: Support for HTTPS on CDN Custom Domains

  • Added a new resource type, CustomDomainHttps, to enable or disable HTTPS on Azure CDN custom domains. This includes support for configuring certificates (CDN-managed or Azure Key Vault) and TLS settings. (provider/pkg/resources/customresources/customresources.go [1] reports/tokenPaths.json [2] sdk/dotnet/Cdn/CustomDomainHttps.cs [3]

Example Implementation

  • Introduced a new example, cdn-custom-domain-https, demonstrating how to enable HTTPS on a CDN custom domain using Pulumi. This includes:

SDK Updates

  • Generated new .NET SDK types to support the CustomDomainHttps resource and its associated configurations:
    • CustomDomainHttps resource definition. (sdk/dotnet/Cdn/CustomDomainHttps.cs sdk/dotnet/Cdn/CustomDomainHttps.csR1-R104)
    • Input and output types for HTTPS configuration and certificate parameters. (sdk/dotnet/Cdn/Inputs/HttpsConfigurationArgs.cs [1] sdk/dotnet/Cdn/Inputs/CdnCertificateSourceParametersArgs.cs [2] sdk/dotnet/Cdn/Inputs/KeyVaultCertificateSourceParametersArgs.cs [3] sdk/dotnet/Cdn/Outputs/HttpsConfiguration.cs [4] sdk/dotnet/Cdn/Outputs/CdnCertificateSourceParameters.cs [5]

Documentation

  • Added a new CLAUDE.md file providing comprehensive guidance for contributors and users of the repository, including build commands, architecture overview, and development workflows. (CLAUDE.md CLAUDE.mdR1-R160)

@github-actions
Copy link

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

@codecov
Copy link

codecov bot commented May 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.08%. Comparing base (750903b) to head (376498f).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4154   +/-   ##
=======================================
  Coverage   58.08%   58.08%           
=======================================
  Files          84       84           
  Lines       13535    13535           
=======================================
  Hits         7862     7862           
  Misses       5081     5081           
  Partials      592      592           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for enabling HTTPS on Azure CDN custom domains by introducing a new resource type, CustomDomainHttps, along with updates to SDKs for Python, Node.js, .NET, and an example demonstrating the usage.

  • Adds CustomDomainHttps resource to enable/disable HTTPS and configure certificates.
  • Updates code generation and token paths to include CustomDomainHttps.
  • Provides an example (cdn-custom-domain-https) illustrating configuring HTTPS on a CDN custom domain.

Reviewed Changes

Copilot reviewed 27 out of 27 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
sdk/python/pulumi_azure_native/init.py Adds CustomDomainHttps to the resource mapping.
sdk/nodejs/types/output.ts & input.ts Introduces new interfaces for HTTPS configuration and certificate parameters.
sdk/nodejs/cdn/customDomainHttps.ts & index.ts Implements the CustomDomainHttps resource and updates resource registration.
sdk/dotnet/Cdn/* Adds generated .NET SDK files for CustomDomainHttps and associated configurations.
reports/tokenPaths.json Updates token paths to include the new CustomDomainHttps endpoint.
provider/pkg/resources/customresources/customresources.go Registers the new cdnCustomDomainHttps custom resource.
examples/cdn-custom-domain-https/* Provides an example demonstrating HTTPS configuration for a CDN custom domain.
CLAUDE.md Adds documentation and contributor guidance updates.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants