Change 19549

tjuricek
tjuricek committed this change into //guest/perforce_software/helix-web-services/main/source
Request Review
Download .zip
Apply a convention for reading models that returns 400 errors when people input garbage on POST/PUT requests.

This is not hit by users of the client SDKs typically, but users of curl, or what have you that mistakenly make a POST request that has no body.
  • Files 28
  • Comments 0
27 edited 1 added 0 deleted
server/src/main/java/com/perforce/hws/server/routes/LoginRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/UsesJsonRequests.java#1
Loading...
server/src/main/java/com/perforce/hws/server/routes/git_fusion/GitFusionCreateRepoConfigRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/git_fusion/GitFusionGetRepoConfigRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/git_fusion/GitFusionUpdateRepoConfigRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceCreateBranchRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceCreateChangeRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceCreateClientRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceCreateDepotRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceCreateGroupRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceCreateJobRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceCreateLabelRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceCreateServerRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceCreateStreamRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceCreateUserRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceUpdateBranchRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceUpdateClientRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceUpdateCounterRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceUpdateDepotRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceUpdateGroupRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceUpdateJobRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceUpdateLabelRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceUpdateProtectionsRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceUpdateServerRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceUpdateStreamRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceUpdateTriggersRoute.java#2
Loading...
server/src/main/java/com/perforce/hws/server/routes/perforce/PerforceUpdateUserRoute.java#2
Loading...
testing/src/main/java/com/perforce/hws/testing/server/AuthTests.java#2
Loading...
Tip: Use n and p to cycle through the changes.