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

Connect to Self-Hosted Hudu instance through CloudFlare ZTNA Tunnel [2/2] #1199

Merged
merged 10 commits into from
Jan 19, 2025

Conversation

Teh-STIG
Copy link
Contributor

See ISSUE#2983 [Feature Request]: Hudu Integration: Self-Hosted instance secured by CloudFlare ZTNA Tunnel and PR#3022

I have modified CIPP-API\Modules\CippExtensions\Public\Hudu\Connect-HuduAPI.ps1 to check if Hudu.CFEnabled checkbox is enabled, if so: add the CloudFlare ClientID/API Key to the connection request to route through the ZTNA tunnel.

This modification is tied to changes to in CIPP\src\data\Extensions.json.

KelvinTegelaar and others added 10 commits August 23, 2024 16:29
Add logic to check if Hudu.CFEnabled checkbox is enabled in from Extension.json**. If so, pull the CloudFlare Client ID and API Key from the cippmsdwl key vault and add them as a header for the initial connection request.

** Hudu.CFEnabled checkbox and input fields were added to /CIPP/blob/main/src/data/Extensions.json
@Teh-STIG
Copy link
Contributor Author

Sorry, to clarify, the [Feature Request] was submitted over on the CIPP repo, not here. (KelvinTegelaar/CIPP#2983). The PR request that creates the Hudu.CFEnabled variable was denied due to the new UI coming out soon however this PR could still be approved for future implementation on the new UI once it is released and I am able to work the checkbox variable and input boxes into the new UI. (KelvinTegelaar/CIPP#3022)

@KelvinTegelaar
Copy link
Owner

Waiting on commiting this one to next major release. Love it!

@KelvinTegelaar KelvinTegelaar merged commit 0ad13cd into KelvinTegelaar:dev Jan 19, 2025
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.

4 participants