Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: puppetlabs/puppetlabs-apache
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main
Choose a base ref
...
head repository: puppetlabs/puppetlabs-apache
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: allow-mod_md-to-reuse-domain
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 1 commit
  • 4 files changed
  • 1 contributor

Commits on Jan 9, 2025

  1. Allow to use the same domain multiple times with mod_md

    When using mod_md to manage TLS certificates, a domain can only appear
    once as a parameter of a MDomain configuration.
    
    When a single node configue multiple Virtual Hosts to serve the same
    website on different IP Addresses or on different ports, and we want to
    use mod_md to manage the TLS certificate, the current code produce a
    MDomain entry in each virtual host, leading to configuration error and
    preventing apache from starting.
    
    This commit rework how the MDomain setting is emitted, and ensure it is
    only output once even if multiple Virtual Hosts configure the same
    domain.
    smortex committed Jan 9, 2025
    Configuration menu
    Copy the full SHA
    58de231 View commit details
    Browse the repository at this point in the history
Loading