Caught this with the new redirect validation logic. The $return_uri was being set as just B123 which is not valid. Prefixing it with / (like is done in HarbormasterBuildActionController already) gives the correct result of reloading the buildable's page.
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Commits
- Restricted Diffusion Commit
rP1ffa16aa6b38: Fix invalid redirect when issuing actions on buildables
Restarted all builds on a buildable, saw the page reload correctly.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- bad-redirect
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 2306 Build 2310: [Placeholder Plan] Wait for 30 Seconds