Closed Bug 960332 Opened 10 years ago Closed 10 years ago

Update Firefox Account server URLs to use the production URLs

Categories

(Firefox :: Sync, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: ckarlof, Assigned: markh)

References

Details

(Whiteboard: [qa+])

Attachments

(1 file, 3 obsolete files)

* firefox.accounts.remoteUrl -> https://accounts.firefox.com/?service=sync&context=fx_desktop
* identity.fxaccounts.auth.uri -> https://api.accounts.firefox.com/v1

A another issue is that we should also make the pref names more consistent.
Issue to make the names more consistent: Bug 960334
Whiteboard: [qa+]
* firefox.accounts.remoteUrl -> https://accounts.firefox.com/?service=sync&context=fx_desktop_v1
* identity.fxaccounts.auth.uri -> https://api.accounts.firefox.com/v1

Updated to version the context.
fxa-content-server PR #296 [1] causes the content server front end to listen for messages from Firefox desktop if firefox.accounts.remoteUrl contains the `context=fx_desktop_v1` search paramater.


[1] - https://github.com/mozilla/fxa-content-server/pull/296/files#diff-4a23ac1286faa3273c8cdc9b4bb5078dR80
Update the URLs
Assignee: nobody → mhammond
Attachment #8364066 - Flags: review?(ckarlof)
This patch also adds a pref for the "settings" URL.
Attachment #8364066 - Attachment is obsolete: true
Attachment #8364066 - Flags: review?(ckarlof)
Attachment #8364069 - Flags: review?(ckarlof)
I presume we should also be matching these on Android.

We currently use:

  public static final String DEFAULT_IDP_ENDPOINT = "https://api-accounts-onepw.dev.lcip.org";
  public static final String DEFAULT_TOKEN_SERVER_ENDPOINT = "http://auth.oldsync.dev.lcip.org";

What should those be?
Status: NEW → ASSIGNED
Summary: Update Firefox Account server URLs to use the production URLS → Update Firefox Account server URLs to use the production URLs
Target Milestone: mozilla29 → ---
Comment on attachment 8364069 [details] [diff] [review]
0012-Bug-960332-update-URLs-to-production-server.patch

Review of attachment 8364069 [details] [diff] [review]:
-----------------------------------------------------------------

Let's not land this immediately. We're not going to update prod FxA until 1/24 (gene on PTO), so it's prudent to remain on the dev box until then.
Attachment #8364069 - Flags: review?(ckarlof) → review+
(In reply to Richard Newman [:rnewman] from comment #6)
> I presume we should also be matching these on Android.
> 
> We currently use:
> 
>   public static final String DEFAULT_IDP_ENDPOINT =
> "https://api-accounts-onepw.dev.lcip.org";
>   public static final String DEFAULT_TOKEN_SERVER_ENDPOINT =
> "http://auth.oldsync.dev.lcip.org";
> 
> What should those be?

Final API/IDP endpoint: https://api.accounts.firefox.com/v1
Final token server endpoint (Desktop change is not covered by this patch): https://token.services.mozilla.com (not currently live)
(In reply to Chris Karlof [:ckarlof] from comment #9)
> Final token server endpoint (Desktop change is not covered by this patch):
> https://token.services.mozilla.com (not currently live)

Sorry, full endpoint is: https://token.services.mozilla.com/1.0/sync/1.5
See bug 963251 for token server URI changes.
Updated against bug 963314.  Carrying r+ forward, but not landing.
Attachment #8364069 - Attachment is obsolete: true
Attachment #8364709 - Flags: review+
oops, screwed up the previous attachment.
Attachment #8364709 - Attachment is obsolete: true
Attachment #8364799 - Flags: review+
Since markh is already dev-ing with this patch applied, let's land at markh's convenience.
Need prod versions of the tos and pp urls in bug 959990
(In reply to Chris Karlof [:ckarlof] from comment #14)
> Since markh is already dev-ing with this patch applied, let's land at
> markh's convenience.

Now's as good a time as any then!

https://hg.mozilla.org/integration/fx-team/rev/464aa919d253
https://hg.mozilla.org/mozilla-central/rev/464aa919d253
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Component: Firefox Sync: Backend → Sync
Product: Mozilla Services → Firefox
Target Milestone: mozilla29 → ---
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: