1352933 - [FlyWeb] Stable naming implementation

Status: RESOLVED WONTFIX

(Core Graveyard :: DOM: Flyweb, defect, P2)

Description

[ron.waisberg]

Attachment: generate_signature.sh

User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36

Steps to reproduce:

Bug for tracking implementation of stable naming for FlyWeb services.

Comment 1

[ron.waisberg]

Attachment: server.js

Comment 2

[ron.waisberg]

Comment on attachment 8853852 [details]
generate_signature.sh

Generates RSA-SHA256 signature that is ASN.1 encoded in the format nsDataSignatureVerifier::VerifyData expects it

Comment 3

[ron.waisberg]

Comment on attachment 8853853 [details]
server.js

Example service implementing authentication flow

Comment 4

[ron.waisberg]

Attachment: stable_naming.patch

Initial implementation of stable naming authentication flow in Gecko.

TODO:
- Refactor and split up AuthenticateService()
- Further testing of error cases
- Sanitize mDisplayName
- Figure out why appending ";flyweb.local" also prepends "www." to the spec
- More flexible signature validation

Comment 5

[ron.waisberg]

Attachment: package.json

Comment 6

[ron.waisberg]

Attachment: public_key.der

Comment 7

[ron.waisberg]

Attachment: private_key.pem

Key pair generated for testing

Comment 8

[ron.waisberg]

Comment on attachment 8853852 [details]
generate_signature.sh

Note - nsDataSignatureVerifier::VerifyData expects the signature to be ASN.1 encoded in this format: https://dxr.mozilla.org/mozilla-central/source/security/manager/ssl/nsDataSignatureVerifier.cpp#30

Currently this script only generates and encodes an RSA-SHA256 signature properly. While it can be easily expanded to other algorithms, it's quite limiting to force a custom ASN.1 structure on the end user.

Comment 9

[Kannan Vijayan [:djvj]]

Comment on attachment 8853854 [details] [diff] [review]
stable_naming.patch

Canceling this as the flyweb code is being removed from trunk.

Comment 10

[Marion Daly [:mdaly]]

Overholt: Patches are up, no reviews. Need to assign someone.

Comment 11

[Justin D'Arcangelo [:justindarc]]

Closing this as we're no longer doing FlyWeb.