[B2G][Helix][sms][zhaotao]hyperlink with chinese characters in sms can not be opened

RESOLVED WONTFIX

Status

P3
normal
RESOLVED WONTFIX
5 years ago
2 years ago

People

(Reporter: lecky.wanglei, Unassigned)

Tracking

unspecified

Firefox Tracking Flags

(tracking-b2g:backlog)

Details

Attachments

(1 attachment, 1 obsolete attachment)

44.00 KB, application/octet-stream
Details
(Reporter)

Description

5 years ago
User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; .NET4.0C; .NET4.0E; aff-kingsoft-ciba; Zune 4.7)

Steps to reproduce:

1.use an android phone to send a sms contains hyperlink with chinese characters to ffos phone
2.tap the hyperlink in ffos phone


Actual results:

in step 2,the hyperlink can not be opened


Expected results:

the hyperlink should be opened successfully.
(Reporter)

Updated

5 years ago
Component: General → Gaia::SMS
Priority: -- → P2
(Reporter)

Updated

5 years ago
blocking-b2g: --- → hd?
Looping Gaia folks.
lecky, it would help a lot if you could provide us a sms db with this, as it's difficult for us european/american to do this.

You just need to "adb pull" the file /data/local/indexedDB/chrome/idb/226660312ssm.sqlite and the directory /data/local/indexedDB/chrome/idb/226660312ssm (the number in the file names might be different). Be careful as when you'll pull the directory it will actually put all the files in your current directory, so better create a new dir locally with that name, cd to that directory, and pull the directory from there.

Thanks !
Flags: needinfo?(lecky.wanglei)
(Reporter)

Updated

5 years ago
Severity: normal → blocker
Flags: needinfo?(lecky.wanglei)
still need the sms db.
Flags: needinfo?(lecky.wanglei)
(Reporter)

Comment 4

5 years ago
hi,julien,sorry for the delay,

please check the attachment file.

the last two format with chinese characters can not be recognized.
Flags: needinfo?(lecky.wanglei)
(Reporter)

Comment 5

5 years ago
Created attachment 798396 [details]
226660312ssm.sqlite
(Reporter)

Comment 6

5 years ago
Created attachment 798397 [details]
226660312ssm.sqlite
Attachment #798396 - Attachment is obsolete: true
(Reporter)

Comment 7

5 years ago
please check the attachment file,

the last three format with chinese characters can not be recognized.
I think the problem is that we use "\w" and this doesn't match extended characters.

I had another algorithm in https://bug846373.bugzilla.mozilla.org/attachment.cgi?id=731827 which did match extended characters. Basically, instead of using "\w" I used "[^\s...]", but I'm afraid this can lead to too many false positives.

Are URL with chinese characters usually prefixed with "http:" so that we can be more strict for those ?
(Reporter)

Comment 9

5 years ago
hi,julien

URL with chinese characters may prefixed with "http://", and may not.

It depends on the end user's using habit.

It will be prefered if both format can be supported.
Joe, can we add this to the backlog please ?
Flags: needinfo?(jcheng)
add to backlog bug 891754
Blocks: 891754
Flags: needinfo?(jcheng)
(Reporter)

Comment 12

5 years ago
hi,wayne,could you check the issue state for me?

add to backlog means will not fix in V1.1?
blocking-b2g: hd? → ---
Flags: needinfo?
(Reporter)

Updated

5 years ago
Flags: needinfo?
(Reporter)

Updated

5 years ago
Flags: needinfo?(wchang)
Lecky,

Correct. Backlogged means for a future version yet to be decided. It will depend on the priority and our available resources.
Severity: blocker → normal
Flags: needinfo?(wchang)
(Reporter)

Updated

5 years ago
Priority: P2 → P3
blocking-b2g: --- → backlog
(Assignee)

Updated

4 years ago
blocking-b2g: backlog → ---
tracking-b2g: --- → backlog
Hey Steve, Bryant,

As you know, we support all URLs that start with http://, but we only recognize URLs with ascii characters and some white-listed TLD when http:// is missing. Do you think it's good enough ? What other platforms are doing ? Is it usual that chinese people write URLs in messages without http:// ?

Thanks !
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(schung)
Flags: needinfo?(bmao)
I seldom(almost never) use the Chinese char in the URL actually, not to mention the URL with Chinese char and without http:// prefix. This kind of URL is valid and works on Android, but I'm not sure if it's an important use case for general user.
Flags: needinfo?(schung)
Move to firefox project. Still, let me know if there are any UX emergency.
Flags: needinfo?(bmao)
Mass closing of Gaia::SMS bugs. End of an era :(
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
Mass closing of Gaia::SMS bugs. End of an era :(
You need to log in before you can comment on or make changes to this bug.