Repository navigation
Add the option to turn off automatic redirect handling - #124
g-host-one wants to merge 4 commits into
Conversation
|
Looks decent, but I'm not a team member (yet?..) :) CI fault is with infra: At dayjob, we had some (cucumber?) publishing issues so ended up using a lockable resource so one report is published at a time... |
|
triage pass: this feature is still missing on master — the redirect strategy is always wired in (restricted to GET/HEAD since #229), there's no opt-out. the ask is still valid. the bad news is the PR predates the HttpClient 5 migration (#197) and doesn't apply cleanly anymore. if you're still around and want to rebase, go ahead. otherwise I'm happy to port it onto current master with you credited — just say something in the next couple of weeks so I know you're interested. — h3nr1-d14z |
|
Thanks for the update and for taking the time to look into this! I’ll review the current state this week and get back to you. |
# Conflicts: # src/main/java/jenkins/plugins/http_request/HttpRequest.java # src/main/java/jenkins/plugins/http_request/HttpRequestExecution.java # src/main/java/jenkins/plugins/http_request/HttpRequestStep.java # src/test/java/jenkins/plugins/http_request/HttpRequestTest.java # src/test/java/jenkins/plugins/http_request/Registers.java
|
@h3nr1-d14z I’ve rebased the PR onto the current master, and all tests are passing. The PR is ready for review. |
Added the option
followRedirectsto turn off automatic redirect handling.Link to relevant issues in GitHub or JiraLink to relevant pull requests, esp. upstream and downstream changes