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

Jump to bottom

Fixes #18855: Make api/status "isAlive" api available from other hosts than localhost #2413

Merged

Normation-Quality-Assistant merged 1 commit into Normation:master from fanf:bug_18855/make_api_status_isalive_api_available_from_other_hosts_than_localhost

Jan 22, 2021

Member

fanf commented Jan 22, 2021 •

edited

https://issues.rudder.io/issues/18855

Two things in that PR:

all API v1 (but "/api/status") are deleted. We can remove all logic in place to taking care of them,
we want to make "/api/status" available from everywhere: remove restriction logic for it.

And since we want to make as little friction as possible for /api/status, we don't even require any version header.


          Fixes #18855: Make api/status \"isAlive\" api available from other ho…

0cbf98c

…sts than localhost

fanf requested a review from amousset

January 22, 2021 15:12

fanf added the Trigger test label

amousset approved these changes

View reviewed changes

Contributor

Normation-Quality-Assistant commented Jan 22, 2021

OK, merging this PR

Normation-Quality-Assistant merged commit 0cbf98c into Normation:master

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