Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Support custom domain names used as OneFuzz endpoints in deployment #2720

Merged
merged 3 commits into from
Jan 5, 2023

Conversation

AdamL-Microsoft
Copy link
Contributor

Summary of the Pull Request

This PR adds support for custom domain names used as OneFuzz endpoints through an optional argument to deploy.py for --custom_domain. Using this argument will include the custom domain passed in the Allowed token audiences setting of the Azure Function App authentication settings.

Fixes #2695

@codecov-commenter
Copy link

codecov-commenter commented Dec 22, 2022

Codecov Report

Merging #2720 (7e0b373) into main (df40c99) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2720   +/-   ##
=======================================
  Coverage   28.75%   28.75%           
=======================================
  Files         320      320           
  Lines       38191    38191           
=======================================
  Hits        10980    10980           
  Misses      27211    27211           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@AdamL-Microsoft AdamL-Microsoft marked this pull request as ready for review January 4, 2023 19:27
Copy link
Contributor

@nharper285 nharper285 left a comment

Choose a reason for hiding this comment

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

Looks great!

@AdamL-Microsoft AdamL-Microsoft merged commit af25752 into main Jan 5, 2023
@AdamL-Microsoft AdamL-Microsoft deleted the allow-custom-domain-names-in-deployments branch January 5, 2023 18:28
@AdamL-Microsoft AdamL-Microsoft mentioned this pull request Jan 5, 2023
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.

Allow custom DNS names as URLs for OneFuzz at deployment
3 participants