-
-
Notifications
You must be signed in to change notification settings - Fork 106
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
Route constructUrl incorrectly builds a URL on a non-standard port #297
Comments
In my opinion, the problem will go down to nette/routing and maybe duplication of #164 |
dg
added a commit
to nette/routing
that referenced
this issue
Dec 4, 2022
dg
added a commit
to nette/routing
that referenced
this issue
Dec 4, 2022
dg
added a commit
to nette/routing
that referenced
this issue
Dec 21, 2022
dg
added a commit
to nette/routing
that referenced
this issue
Jan 9, 2023
dg
added a commit
to nette/routing
that referenced
this issue
Jan 9, 2023
dg
added a commit
to nette/routing
that referenced
this issue
Jan 10, 2023
dg
added a commit
to nette/routing
that referenced
this issue
Jan 10, 2023
dg
added a commit
to nette/routing
that referenced
this issue
Jan 10, 2023
dg
added a commit
to nette/routing
that referenced
this issue
Jan 10, 2023
dg
added a commit
to nette/routing
that referenced
this issue
Jan 10, 2023
dg
added a commit
to nette/routing
that referenced
this issue
Jan 18, 2023
dg
added a commit
to nette/union
that referenced
this issue
Feb 7, 2023
dg
added a commit
to nette/routing
that referenced
this issue
Jul 30, 2023
dg
added a commit
to nette/routing
that referenced
this issue
Aug 5, 2023
dg
added a commit
to nette/routing
that referenced
this issue
Sep 26, 2023
dg
added a commit
to nette/union
that referenced
this issue
Sep 26, 2023
dg
added a commit
to nette/routing
that referenced
this issue
Oct 5, 2023
dg
added a commit
to nette/routing
that referenced
this issue
Oct 9, 2023
dg
added a commit
to nette/union
that referenced
this issue
Oct 9, 2023
dg
added a commit
to nette/union
that referenced
this issue
Oct 17, 2023
dg
added a commit
to nette/union
that referenced
this issue
Oct 25, 2023
dg
added a commit
to nette/union
that referenced
this issue
Nov 2, 2023
dg
added a commit
to nette/routing
that referenced
this issue
Nov 13, 2023
dg
added a commit
to nette/routing
that referenced
this issue
Dec 11, 2023
dg
added a commit
to nette/routing
that referenced
this issue
Dec 11, 2023
dg
added a commit
to nette/routing
that referenced
this issue
Jan 21, 2024
dg
added a commit
to nette/routing
that referenced
this issue
Jan 21, 2024
dg
added a commit
to nette/routing
that referenced
this issue
May 2, 2024
dg
added a commit
to nette/union
that referenced
this issue
May 28, 2024
dg
added a commit
to nette/routing
that referenced
this issue
Jun 18, 2024
dg
added a commit
to nette/routing
that referenced
this issue
Nov 4, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Version: 3.1.4
Bug Description
If a port other than the standard port is used, Router::constructUrl returns a URL without a non-standard port in the case of an absolute route.
Steps To Reproduce
Start application on other port than 80 or 443.
Use Route
//%domain%/<presenter>/<action>[/<id>]
Expected Behavior
Route::constructUrl returns a url with the same port as the one in the request
The text was updated successfully, but these errors were encountered: