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

Bug: NotFound handler was called although route exists #97

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sknr
Copy link

@sknr sknr commented Jan 15, 2024

πŸ“„ What has changed

There was a bug in the _findRoute method of the file node.go. Please see the adjusted test TestMethodNotAllowed in router_test.go as an example.

βœ… Before submitting, please be sure to check this list

  • your branch will not cause merge conflict with main branch
  • unit tests are included
  • the CHANGELOG has been updated
  • you are not committing extraneous files or sensitive data

@sknr sknr force-pushed the bugfix/handle-missing-case branch 2 times, most recently from 5dcb9bd to a394720 Compare January 15, 2024 21:03
@sknr
Copy link
Author

sknr commented Jan 23, 2024

Hi @vmihailenco, is this project still actively maintained? If yes, it would be nice if someone could have a look at my PR.

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.

None yet

1 participant