Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Support following HTTP redirects encountered during notification POSTS #4
Ideally I'd like to add a configuration option to allow users to opt-in to redirect following but a few hours trying to work through jelly binding and the hudson form taglib left me stumped. I'm not seeing any obvious harm or breakages having 307 following on by default would cause, but please push back if this is misguided.
This comment has been minimized.
This comment has been minimized.Show comment Hide comment
Space/tab codestyle varied from file to file and within files so I stuck with local convention in Protocol.java near the lines I edited. Since the test was a rewrite its consistently using 2 space indents, 4 space line continuations. Let me know if there is a preferred style or style guide and I can adapt.