Skip to content
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

feat(BizTypeEnum): add HttpMethod check #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jameslahm
Copy link

@jameslahm jameslahm commented Apr 27, 2021

目前在Http处理中Controller好像是不能拿到请求方法的,感觉可以对BizTypeEnum加上对Http请求方法的限制?

@xuwm14
Copy link
Owner

xuwm14 commented Apr 27, 2021

目前在Http处理中Controller好像是不能拿到请求方法的,感觉可以对BizTypeEnum加上对Http请求方法的限制?

是的,不能拿到请求方法,且没有限制请求方法,但是GET和POST方式请求的参数都可以被解析出来的,暂时没有限制请求方法的必要吧。

@xuwm14
Copy link
Owner

xuwm14 commented Apr 27, 2021

目前在Http处理中Controller好像是不能拿到请求方法的,感觉可以对BizTypeEnum加上对Http请求方法的限制?

不过你加上也可以,这个没有问题,是对框架的一大完善!很好!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants