[MULTISITE-180] Default port is added for https links Created: 28/Oct/23 Updated: 10/Jan/24 Resolved: 03/Nov/23 |
|
| Status: | Closed |
| Project: | Magnolia Multisite Module |
| Component/s: | None |
| Affects Version/s: | 2.1.2 |
| Fix Version/s: | 2.1.3, 3.0.0 |
| Type: | Bug | Priority: | Neutral |
| Reporter: | Philipp Gaschuetz | Assignee: | Jaroslav Simak |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Σ Remaining Estimate: | Not Specified | Remaining Estimate: | Not Specified |
| Σ Time Spent: | Not Specified | Time Spent: | Not Specified |
| Σ Original Estimate: | Not Specified | Original Estimate: | Not Specified |
| Issue Links: |
|
|||||||||||||||||||||||||
| Sub-Tasks: |
|
|||||||||||||||||||||||||
| Template: | ||||||||||||||||||||||||||
| Acceptance criteria: |
Empty
|
|||||||||||||||||||||||||
| Task DoD: |
[X]*
Doc/release notes changes? Comment present?
[X]*
Downstream builds green?
[X]*
Solution information and context easily available?
[X]*
Tests
[X]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
|||||||||||||||||||||||||
| Bug DoR: |
[X]*
Steps to reproduce, expected, and actual results filled
[X]*
Affected version filled
|
|||||||||||||||||||||||||
| Release notes required: |
Yes
|
|||||||||||||||||||||||||
| Date of First Response: | ||||||||||||||||||||||||||
| Epic Link: | Support | |||||||||||||||||||||||||
| Sprint: | DevX 50 | |||||||||||||||||||||||||
| Team: | ||||||||||||||||||||||||||
| Work Started: | ||||||||||||||||||||||||||
| Description |
|
A pull request for this issue has been created at https://git.magnolia-cms.com/projects/ENTERPRISE/repos/multisite/pull-requests/79/overview
info.magnolia.multisite.DomainNamePathTransformer is used to create fully qualified links. The current implementation unnecessarily appends the default port number (443) for https links Steps to reproduce
Expected resultsit should return i.e. https://example.com Actual resultsreturns i.e. https://example.com:443 Workaroundn/a Development notesThe current implementation makes erroneous assumptions. Tests have been added. |