Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Two regex fixes #2

Merged
merged 2 commits into from Dec 31, 2011

Conversation

Projects
None yet
2 participants
Contributor

emmausrs commented Dec 31, 2011

  1. Links beginning with double slashes (e.g. //example.com/) weren't anonymised.
  2. The regular expression was missing a ^, causing the link to be anonymised if it contained http(s):// anywhere within the URL.

savetheinternet added a commit that referenced this pull request Dec 31, 2011

@savetheinternet savetheinternet merged commit a12a83e into savetheinternet:master Dec 31, 2011

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