-
Notifications
You must be signed in to change notification settings - Fork 395
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
Revert "config: Add a flag to control the /metrics and /status endpoints" #3251
Revert "config: Add a flag to control the /metrics and /status endpoints" #3251
Conversation
sigh i hate the 1.x code so much... Please sign off the commit. |
df78d8f
to
090bb8f
Compare
…nts" This reverts commit d10ead4. Signed-off-by: Stéphane Maniaci <stephane.maniaci@beta.gouv.fr>
090bb8f
to
b79e09e
Compare
Done. Apologies for not noticing this beforehand, next time I'll make sure our changesets get tested in our own environments first. |
It's okay. There is a reason why we're rewriting this unpredictable mess 🙃 |
Tbh: I don't think that it's necessary to revert EVERYTHING. Just revert the code for deactivation of the status route and keep the code that deactivates the metrics route because the metrics route is the sensible one. |
I've merged this PR to repair the master branch. If you still want to deactivate the metric route then feel free to create a new PR. |
Sorry I was busy this weekend, I'll push a new PR soon. Thanks for taking care of this! |
Reverts #3219
Apologies there's a bug in the PR above. Hedgedoc uses the
/config
endpoint to transmit some information client-side, via writing into thewindow
object.Though I don't think it's a great mechanism, I don't have a patch yet to work around it, so revert the previous PR in the meantime.