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.

Sign up for GitHub

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

Jump to bottom

Possible problem with '/' in Controller with [MVCPath('/')] #444

Closed
TomsGoe opened this issue Oct 29, 2020 · 1 comment
Closed

Possible problem with '/' in Controller with [MVCPath('/')] #444

TomsGoe opened this issue Oct 29, 2020 · 1 comment
Assignees
@danieleteti
Labels
accepted Issue has been accepted and inserted in a future milestone
Milestone
3.2.1-carbon

Comments

@TomsGoe
Copy link

TomsGoe commented Oct 29, 2020

If you use the example "plainclient" and "plainserver" from the framework, you always get the error message "not found". It seems that the search for the controller implemented for the URL does not work correctly.
After swapping the blocks in MVCFramework.Router.pas the problem seems to be solved. I ask for verification.
2020-10-29 13_02_26-jwtplainclient - RAD Studio 10 1 Berlin - MVCFramework Router
@danieleteti danieleteti self-assigned this Nov 27, 2020
@danieleteti danieleteti added the accepted Issue has been accepted and inserted in a future milestone label Nov 27, 2020
@danieleteti danieleteti added this to the 3.2.1-carbon milestone Nov 27, 2020
danieleteti added a commit that referenced this issue Nov 27, 2020
@danieleteti
https://github.com/danieleteti/delphimvcframework/issues/444
ce3f0ec
@allanwolski
Copy link

I have the same issue here in master.

[MVCPath('/')]
TDocsController = class(TMVCController)
public
...

DelphiMVCFramework
HTTP 404: [EMVCException] Not Found

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@danieleteti danieleteti

Labels
accepted Issue has been accepted and inserted in a future milestone
Projects
None yet
Milestone
3.2.1-carbon
Development

No branches or pull requests
3 participants
@danieleteti @allanwolski @TomsGoe