Closed Bug 1689290 Opened 4 years ago Closed 4 years ago

The "single-site" link handler should be able to handler IP addresses

Categories

(Thunderbird :: General, defect)

defect

Tracking

(thunderbird_esr78 unaffected, thunderbird86+ fixed)

RESOLVED FIXED
87 Branch
Tracking Status
thunderbird_esr78 --- unaffected
thunderbird86 + fixed

People

(Reporter: darktrojan, Assigned: darktrojan)

Details

Attachments

(1 file)

The same-site link handler uses the eTLD service to determine if two URL are from the same site, but this doesn't handle IP addresses. If it's the same IP, it should be considered the same site.

I mean single-site. I really should know what I called things.

Summary: The "same-site" link handler should be able to handler IP addresses → The "single-site" link handler should be able to handler IP addresses

That should do the trick. I've started a Try run to see if I've managed to break anything else: https://treeherder.mozilla.org/jobs?repo=try-comm-central&revision=f48b940da74dbcff05d85edee2bf13571fc36172

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/bf3be1f71ed5
Fix handling of IP addresses in the "single-site" link handler. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 87 Branch

Comment on attachment 9199734 [details]
Bug 1689290 - Fix handling of IP addresses in the "single-site" link handler. r?mkmelin

[Approval Request Comment]
Regression caused by (bug #):
User impact if declined: not a major problem but this does affect a project Magnus is working on
Testing completed (on c-c, etc.): on c-c 5 days
Risk to taking this patch (and alternatives if risky): none

Attachment #9199734 - Flags: approval-comm-beta?

Comment on attachment 9199734 [details]
Bug 1689290 - Fix handling of IP addresses in the "single-site" link handler. r?mkmelin

[Triage Comment]
Approved for beta

Attachment #9199734 - Flags: approval-comm-beta? → approval-comm-beta+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: