wbrawner/thunderbird-android
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #2519 from k9mail/fix_linkifying_of_https_uris

Browse source 
Fix linkify issue with https: and rtsp: URIs
This commit is contained in:
cketti 2017-05-02 04:20:18 +02:00 committed by GitHub
commit cc2c242c1f
2 changed files with 12 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
k9mail/src/main/java/com/fsck/k9/message/html/HttpUriParser.java
View file

@ -30,11 +30,11 @@ class HttpUriParser implements UriParser {
// Scheme
String shortScheme = text.substring(currentPos, Math.min(currentPos + 7, text.length()));
String longScheme = text.substring(currentPos, Math.min(currentPos + 8, text.length()));
if (shortScheme.equalsIgnoreCase("https://")) {
if (longScheme.equalsIgnoreCase("https://")) {
currentPos += "https://".length();
} else if (shortScheme.equalsIgnoreCase("http://")) {
currentPos += "http://".length();
} else if (longScheme.equalsIgnoreCase("rtsp://")) {
} else if (shortScheme.equalsIgnoreCase("rtsp://")) {
currentPos += "rtsp://".length();
} else {
return startPos;

10
k9mail/src/test/java/com/fsck/k9/message/html/HttpUriParserTest.java
View file

@ -27,6 +27,16 @@ public class HttpUriParserTest {
assertLinkify("http://www.google.com");
}
@Test
public void simpleDomainWithHttps() {
assertLinkify("https://www.google.com");
}
@Test
public void simpleRtspUri() {
assertLinkify("rtsp://example.com/media.mp4");
}
@Test
public void invalidDomainIgnored() {
assertLinkIgnored("http://-www.google.com");