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

[TASK] Include path to compare if request and target uri are the same #38

Closed
wants to merge 1 commit into from
Closed

[TASK] Include path to compare if request and target uri are the same #38

wants to merge 1 commit into from

Conversation

gmartino27
Copy link

Related: #37

@lochmueller
Copy link
Owner

Hey @gmartino27
could you check the tests?
If you push a new change set to your branch, the test will executed again.
Regards,
Tim

@lochmueller
Copy link
Owner

Close this PR. Adding the "path" to the "buildRedirectUri" function, will result in a target uri incl. the path. But the path should be checked in the checkUri function and not added to the real target of the redirect. I will fix this shortly.

@lochmueller lochmueller closed this Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gmartino27 @lochmueller