Tapestry
  1. Tapestry
  2. TAPESTRY-1449

Leave off port 443 when generating https URLs

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 4.1.2
    • Fix Version/s: 4.1.2
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      Tapestry 4.1.2 snapshot from 5-8, WinXP, Firefox 2

      Description

      https has a default port of 443. When Tapestry generates absolute URLs, it should leave off the ":443" if the port is 443 and https is being used.

      1. patch.txt
        0.8 kB
        Ben Dotte
      2. enginelinkpatch.txt
        1.0 kB
        Ben Dotte

        Activity

        Ben Dotte created issue -
        Hide
        Ben Dotte added a comment -

        Patch is attached against the 5-8 snapshot.

        Show
        Ben Dotte added a comment - Patch is attached against the 5-8 snapshot.
        Ben Dotte made changes -
        Field Original Value New Value
        Attachment patch.txt [ 12356934 ]
        Hide
        Ben Dotte added a comment -

        Add patch to leave off :443 on https URLs for engine service links

        Show
        Ben Dotte added a comment - Add patch to leave off :443 on https URLs for engine service links
        Ben Dotte made changes -
        Attachment enginelinkpatch.txt [ 12356935 ]
        Hide
        Jesse Kuhnert added a comment -

        Patches applied, thanks.

        Show
        Jesse Kuhnert added a comment - Patches applied, thanks.
        Jesse Kuhnert made changes -
        Fix Version/s 4.1.2 [ 12312202 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Assignee Jesse Kuhnert [ jkuhnert ]
        Mark Thomas made changes -
        Workflow jira [ 12403498 ] Default workflow, editable Closed status [ 12569144 ]
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12569144 ] jira [ 12590408 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        4d 5h 29m 1 Jesse Kuhnert 13/May/07 01:42

          People

          • Assignee:
            Jesse Kuhnert
            Reporter:
            Ben Dotte
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development