Skip to content

New management plugin option: opt-in to serve the HTTP API reference page only to authenticated users (backport #13698) #13714

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

Merged
merged 2 commits into from
Apr 9, 2025

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Apr 9, 2025

This allows restricting access to the /api/index.html and the /cli/index.html page to authenticated users should the user really want to.

Tested manually. I don't really want to add tests for such a weird feature. :-) But I guess I can if someone insists.

This should be backported all the way to 3.13.


This is an automatic backport of pull request #13698 done by Mergify.

lhoguin and others added 2 commits April 9, 2025 06:32
This allows restricting access to the /api/index.html and
the /cli/index.html page to authenticated users should the
user really want to. This can be enabled via advanced.config.

(cherry picked from commit 400e800)
@mergify mergify bot assigned lhoguin Apr 9, 2025
@mergify mergify bot added the make label Apr 9, 2025
@michaelklishin michaelklishin merged commit 88c7ad6 into v4.1.x Apr 9, 2025
268 of 271 checks passed
@michaelklishin michaelklishin deleted the mergify/bp/v4.1.x/pr-13698 branch April 9, 2025 06:47
@michaelklishin michaelklishin changed the title Add new option require_auth_for_api_desc_page to mgmt (backport #13698) New management plugin option: opt-in to serve the HTTP API reference page only to authenticated users (backport #13698) Apr 9, 2025
michaelklishin added a commit that referenced this pull request Apr 9, 2025
New management plugin option: opt-in to serve the HTTP API reference page only to authenticated users (backport #13698) (backport #13714)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants