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

Remote access API on every non-ci.j.io instance #3045

Closed
daniel-beck opened this issue Jul 11, 2022 · 6 comments · Fixed by jenkins-infra/jenkins-infra#2270
Closed

Remote access API on every non-ci.j.io instance #3045

daniel-beck opened this issue Jul 11, 2022 · 6 comments · Fixed by jenkins-infra/jenkins-infra#2270

Comments

@daniel-beck
Copy link

Service(s)

cert.ci.jenkins.io, trusted.ci.jenkins.io

Summary

Could every private instance have the remote API accessible? 🥺

With authentication required on those instances, and Basic authenticated requests allowed by the reverse proxy rule, the only difference is interactive use, which I attempt occasionally. Not a blocker, but just one of those behaviors improperly applied to private instances because ci.j.io had it.

release.ci and infra.ci are unaffected (Kubernetes special?) but trusted.ci and cert.ci are.

Reproduction steps

No response

@lemeurherve
Copy link
Member

We'll disable ci.jenkins.io, trusted.ci.jenkins.io and cert.ci.jenkins.io for the next minutes in order to update their configurations.

@lemeurherve
Copy link
Member

The remote access API has been unblocked on cert.ci.jenkins.io and trusted.ci.jenkins.io

@daniel-beck
Copy link
Author

@lemeurherve Doesn't work for me. Going to https://cert.ci.jenkins.io/api/xml still redirects me to https://www.jenkins.io/infra/ci-redirects/

@daniel-beck daniel-beck reopened this Jul 12, 2022
@lemeurherve
Copy link
Member

Damn, there is another directive redirecting api/xml instead of "emptying" it: https://github.com/jenkins-infra/jenkins-infra/blob/production/dist/profile/manifests/jenkinscontroller.pp#L352-L353

Preparing a pull request to fix it.

@lemeurherve
Copy link
Member

@daniel-beck
Copy link
Author

Thanks! Used it today for a purpose other than testing already.

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

Successfully merging a pull request may close this issue.

3 participants