Skip to content
This repository was archived by the owner on Nov 6, 2023. It is now read-only.
This repository was archived by the owner on Nov 6, 2023. It is now read-only.

Rewriting of URLs with fixed port number causes trouble on sqlite.org #1461

Closed

Description

@hannob

Have a look at this page:
https://www.sqlite.org/src/wiki?name=Bug+Reports

There is a link to:
http://sqlite.org:8080/cgi-bin/mailman/listinfo/sqlite-users

As HTTS everywhere has sqlite.org in its database it will try to rewrite the URL. However this URL has a port number - and of course there can't be an HTTPS and an HTTP server on the same port at the same time. So the rewriting will cause the request to fail.

HTTPS Everywhere probably should not rewrite URLs with a port number (except when it knows that there is both an http and an https server on uncommon ports - but in that case it'd need to know which port the https one is, so this can only be whitelistet).

Activity

  1. reedy commented on Apr 16, 2015

    @reedy
    Contributor

    I wonder if we can "easily" fix this by just explicitly excluding the sqlite mailman setup in the sqlite ruleset?

    I know this probably needs looking at in a larger sense, but this improves the situation to begin with

  2. jsha commented on Apr 17, 2015

    @jsha
    Member

    Yes, let's add a mailman exclusion for now.

    This is a problem introduced by my recent encouragement of the http->https simple rules. Previously we would have had:

    <rule from="^http://sqlite.org/" to="https://sqlite.org/" />
    

    Since the slash was explicitly part of the match it wouldn't have matched anything with a port number. I think the right fix is for matching to punt on any URL whose port is not 80 or 443.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions