1047521 - Set up a way for people to create webmaker accounts from welcome.wmo

Status: RESOLVED FIXED

(Webmaker Graveyard :: webmaker.org, defect)

Description

[Aki Rose Braun [:akirose]]

Bring it on, persona.

Maybe we can do this with a thin proxy on wmo that welcome hits before triggering the login function? Or can we do it straight from welcome.wmo because it shares the same domain?

Comment 1

[Adam Lofting (:adamlofting)]

@Cade

If we use https://github.com/mozilla/webmaker-auth-client 

Does this always need it's own local version of webmaker-auth running, or is there a way these static landing pages would be able to talk to the existing instance running for webmaker.org?

These landing pages are currently from a static generator, so the easiest solution would be entirely client-side.

Comment 2

[Adam Lofting (:adamlofting)]

Also just making note here while I remember. 

If we implement webmaker-auth-client on these pages, we should remove webmaker-rid.js from them as it has duplicate functionality.

Comment 3

[Aki Rose Braun [:akirose]]

Honestly in a perfect world, the "Sign Up" and "Log In" button/links on welcome.wmo would just go to sign up/login GET routes on wmo (triggering persona modals for now, rendering signup/login forms/options later?), but idk how realistic that is.

It looks like the webmaker-auth-client requires the node side (csrf token?), so I'm not totally certain where to go with this next. Some kind of thin proxy on wmo I think.

Comment 4

[Adam Lofting (:adamlofting)]

Just adding a see also: Bug 1018237 << this was a similar request before the landing page work started. It can probably be closed as a duplicate if we solve this one.

Comment 5

[Chris DeCairos (:cade)]

Auth is going to require a proxy living somewhere (likely webmaker.org).

The problems I foresee with these static pages are getting CSRF tokens for the authenticate/create user APIs and enabling CORS on the static domains for the proxy routes already living on Webmaker.org.

Comment 6

[Adam Lofting (:adamlofting)]

Adding a requirement to this piece of work (it might be there by default):

The login process on these pages should allow us to specify a 'next URL' for the user. 

So that after someone creates an account we can route them into an appropriate first action, whether that's a starter make or something else entirely.

Comment 7

[Aki Rose Braun [:akirose]]

This is definitely going to require a new EC2 instance. It will run an express server (which hopefully we can keep as bare-bones as possible) and ideally have incredibly high availability, as it will be accepting snippet traffic. Additionally, we'll need to transition the static site over from S3. Cade, are you the person to help me set that up in AWS?

Comment 8

[Chris DeCairos (:cade)]

I'm not familiar enough with AWS to set this up, we'll need JP.

Wouldn't moving this off of a static page defeat the purpose of having it there in the first place? 

We could implement a route for the static domain that will serve it a CSRF token to use in subsequent requests to the webmaker.org login proxy routes. We'd also have to add some CORS headers to those routes to allow welcome.wm.org.

Comment 9

[Jon Buckley [:jbuck]]

Instead of setting up a new static server, why not take advantage of CloudFront's multiple origin support and setup the login server as this additional origin? I'm not sure what the local dev story would be, but it should work well for staging and production.

Comment 10

[Aki Rose Braun [:akirose]]

Jbuck you have my attention. I'd LOVE to keep this all on a static site, but webmaker auth is gonna be very important and seemingly complicates everything.

I'm not familiar with multiple origins on CF, but if you (or JP) could help me understand how staging/prod would work, maybe I could fiddle with the express server used for local dev?

Comment 11

[Jon Buckley [:jbuck]]

Aki and I had a chat today and decided upon the following strategy:

1) Build webmaker-auth server for the webmaker-landing-pages staging and production
2) Integrate the webmaker-auth module into the webmaker-landing-pages server for local dev
3) Configure webmaker-landing-pages CDN to support webmaker login

Bugs are filed. Lets get to it!

Comment 12

[Aki Rose Braun [:akirose]]

I know I have 2 & you have 3... is 1 this bug? And if so can I reassign it to someone more qualified?

Comment 13

[Aki Rose Braun [:akirose]]

Just kidding, I'm an idiot.

I should note, from Comment #11

1. is Bug #1052019
2. is Bug #1052017
3. is Bug #1052018

Comment 14

[Aki Rose Braun [:akirose]]

Attachment: https://github.com/mozilla/webmaker-landing-pages/pull/54

Comment 15

[Aki Rose Braun [:akirose]]

Comment on attachment 8473222 [details] [review]
https://github.com/mozilla/webmaker-landing-pages/pull/54

Ricardo: This PR can be tested by going to <http://localhost:9006/signup/mentors> and/or <http://localhost:9006/signup/learners>. Simply being able to log in and create an account should be considered "right" for this PR. You should still wind up at the same page after you're done.

Comment 16

[Aki Rose Braun [:akirose]]

Comment on attachment 8473222 [details] [review]
https://github.com/mozilla/webmaker-landing-pages/pull/54

Adam: I just have one question for you on this PR (though you're welcome to review/comment on the whole thing, I just recognize it's huge) on github.

jbuck: This one is huge too. Ricardo is doing the full review, just looking for a quick skim of login.js in relation to the PR you R+ed yesterday, and a general gut-check from you if that's okay.

Comment 17

[Aki Rose Braun [:akirose]]

Attachment: https://github.com/mozilla/webmaker-landing-pages/pull/56

Comment 18

[Adam Lofting (:adamlofting)]

Comment on attachment 8473222 [details] [review]
https://github.com/mozilla/webmaker-landing-pages/pull/54

I didn't know the answer to that question, so I've tagged jbuck in the PR comment.

I also asked a quick Q about window.location.origin.

Comment 19

[Ricardo Vazquez [:ricardo]]

Aki, I'm not able to successfully run `grunt dev` on your branch. This is the screenshot of the output (https://www.dropbox.com/s/uoi7jb250snq42s/Screenshot%202014-08-18%2011.41.03.png). I also noticed the build is failing, could that be also part of the problem?

Comment 20

[Jon Buckley [:jbuck]]

Comment on attachment 8473222 [details] [review]
https://github.com/mozilla/webmaker-landing-pages/pull/54

I added some comments to the PR, but it's looking pretty solid!

Comment 21

[Aki Rose Braun [:akirose]]

Comment on attachment 8473222 [details] [review]
https://github.com/mozilla/webmaker-landing-pages/pull/54

I'm so sorry, Ricardo. That was an oversight on my part of a couple of things...

this is everything i forgot to do

- update the README (fixed)
- warn you that you'd need to copy the env.dist file to .env
- give the heads-up to run npm install again
- direct you towards https://github.com/mozilla/login.webmaker.org to clone and run the login server before testing my branch

Comment 22

[[github robot]]

Commits pushed to master at https://github.com/mozilla/webmaker-landing-pages

https://github.com/mozilla/webmaker-landing-pages/commit/58cf6f998c3e415d69782c30f813543385750f59
Bug 1047521 - Set up a way for people to create webmaker accounts from welcome.wmo

https://github.com/mozilla/webmaker-landing-pages/commit/5d37adef6ce8272839226e8e99c45036189cc9cf
Merge pull request #54 from gesa/1047521/create-account-front-end

Bug 1047521 - Set up a way for people to create webmaker accounts from welcome.wmo

Comment 23

[Ricardo Vazquez [:ricardo]]

Boom! Everything redirects as expected

Comment 24

[[github robot]]

Commits pushed to master at https://github.com/mozilla/webmaker-landing-pages

https://github.com/mozilla/webmaker-landing-pages/commit/9cd645e7272dc4ea4daf940ae95f2a2efca41853
Bug 1047521 - Signup/Login redirects

https://github.com/mozilla/webmaker-landing-pages/commit/9aa59d0dbed70b619650cfd723a4e2d36a74ee14
Merge pull request #56 from gesa/1047521/next-step

Bug 1047521 - welcome.wmo redirects

Comment 25

[Aki Rose Braun [:akirose]]

IT WORKS! ^_^ thanks for all your help jbuck, ricardo, adam, & cade!