Open Bug 1633710 Opened 5 months ago Updated 24 days ago

Replace nsIPrincipal.uri functions with new Principal counterparts in js

Categories

(Core :: DOM: Security, task, P2)

task

Tracking

()

REOPENED
mozilla78
Tracking Status
firefox78 --- affected

People

(Reporter: sstreich, Assigned: sstreich, NeedInfo)

References

(Blocks 1 open bug)

Details

(Keywords: leave-open, Whiteboard: [domsecurity-active])

Attachments

(21 files, 4 obsolete files)

47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review

See Metabug :)

  • As we now have a solid set of functions of uri in principal, so we should move the callees to principal.
Assignee: nobody → sstreich
Status: NEW → ASSIGNED
Priority: -- → P2
Whiteboard: [domsecurity-active]
Pushed by malexandru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a1339bfb947e
Move js uri.hostPort to principal r=ckerschb
https://hg.mozilla.org/integration/autoland/rev/5384d06f6664
Move js uri.schemeIs to principal r=ckerschb
https://hg.mozilla.org/integration/autoland/rev/e5cf1b1ebf93
Move js uri.equals to principal r=ckerschb
Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla78
Status: RESOLVED → REOPENED
Keywords: leave-open
Resolution: FIXED → ---
Pushed by rmaries@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7c728e75474e
Move js uri.AsciiHost to principal r=ckerschb
Pushed by rmaries@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/461c861e1190
principal.URI in Sanitizer.jsm r=ckerschb,johannh

Sorry about that, fixed the issue and updated the patch :)

Flags: needinfo?(sstreich)
Pushed by abutkovits@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/114eee40b1dc
principal.URI in Sanitizer.jsm r=ckerschb,johannh
Pushed by dluca@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2c46bcd325bb
Move js uri.host to principal r=ckerschb,preferences-reviewers,ntim
Pushed by dluca@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/255d9078dd17
Move js uri.host to principal r=ckerschb,preferences-reviewers,ntim
Flags: needinfo?(sstreich)
Pushed by abutkovits@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/da35c25c966f
Move js uri.host to principal r=ckerschb,preferences-reviewers,ntim
Attachment #9162491 - Attachment description: Bug 1633710 - Move Add createContentPrincipal to nsIPrincipal r=ckerschb → Bug 1633710 - Remove createContentPrincipal callers with nsIPrincipal.uri r=ckerschb

Comment on attachment 9162488 [details]
Bug 1633710 - Refactor SiteDataManager.jsm to not use principal.uri r=ckerschb

Revision D82912 was moved to bug 1654583. Setting attachment 9162488 [details] to obsolete.

Attachment #9162488 - Attachment is obsolete: true

Comment on attachment 9162493 [details]
Bug 1633710 - Remove principal.uri in RemotePageAccessManager.jsm r=ckerschb

Revision D82917 was moved to bug 1655945. Setting attachment 9162493 [details] to obsolete.

Attachment #9162493 - Attachment is obsolete: true
Pushed by ncsoregi@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4ed7b13fb7a5
Move uri.spec calls to nsIPrincipal r=necko-reviewers,geckoview-reviewers,preferences-reviewers,ckerschb,ntim,snorp,valentin

Sorry about that, test is green now - :)

Flags: needinfo?(sstreich)
Pushed by cbrindusan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ff5cb90ca89f
Move uri.spec calls to nsIPrincipal r=necko-reviewers,geckoview-reviewers,preferences-reviewers,ckerschb,ntim,snorp,valentin
Pushed by abutkovits@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6d39aab86966
Move display-(spec/prepath/host) to nsIPrincipal r=geckoview-reviewers,snorp
Pushed by nbeleuzu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/07e51bfc3436
Replace principal.uri.prepath with principal.prepath r=ckerschb

Comment on attachment 9162495 [details]
Bug 1633710 - Add isLocalIpAdress to nsIPrincipal r=ckerschb

Revision D82919 was moved to bug 1660822. Setting attachment 9162495 [details] to obsolete.

Attachment #9162495 - Attachment is obsolete: true

Comment on attachment 9162494 [details]
Bug 1633710 - Remove principal.uri in BrowserUtils.jsm r=ckerschb

Revision D82918 was moved to bug 1661280. Setting attachment 9162494 [details] to obsolete.

Attachment #9162494 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.