-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
1.0.3 modified api #1004
1.0.3 modified api #1004
Conversation
in one file
in datasourcemanager package
return Message.ok().data("update_id", envId); | ||
} | ||
|
||
@RequestMapping(value = "/env/{env_id}/form",method = RequestMethod.PUT,headers = MediaType.MULTIPART_FORM_DATA) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
heders or consumes?
javas.ws.rs.core.MediaType can be replaced by org.springframework.http.MediaType
@POST | ||
@Path("/env/json") | ||
public Response insertJsonEnv(DataSourceEnv dataSourceEnv, @Context HttpServletRequest req){ | ||
return RestfulApiHelper.doAndResponse(()->{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
About RestfulApiHelper.doAndResponse
may be modify RestfulApiHelper.doAndResponse return object of Response to Message will be better
in datasourcemanager package
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
What is the purpose of the change
it can transform jersey to spring mvc api
Brief change log
remove jersey api
add spring mvc api
imcomplete part
but i can not transform requestbody in jersey to spring mvc .
just like FormDataMultiPart multiPartForm in RequestBody