Skip to content

G
gitlab-ce
Switch branch/tag
  1. 06 Jul, 2017 4 commits
    • Timothy Andrew's avatar
      Move OAuth "remember me" javascript logic into a class. · d705a254
      Timothy Andrew authored
      d705a254
    • Timothy Andrew's avatar
      Test the "Remember Me" flow for OAuth-based login. · 43337c12
      Timothy Andrew authored
      43337c12
    • Timothy Andrew's avatar
      Add integration tests around OAuth login. · e936db96
      Timothy Andrew authored 
      - There was previously a test for `saml` login in `login_spec`, but this didn't
  seem to be passing. A lot of things didn't seem right here, and I suspect that
  this test hasn't been running. I'll investigate this further.

- It took almost a whole working day to figure out this line:

    OmniAuth.config.full_host = ->(request) { request['REQUEST_URI'].sub(request['REQUEST_PATH'], '') }

  As always, it's obvious in retrospect, but it took some digging to figure out
  tests were failing and returning 404s during the callback phase.

- Test all OAuth providers - github, twitter, bitbucket, gitlab, google, and facebook
      e936db96
    • Timothy Andrew's avatar
      Implement "remember me" for OAuth-based login. · 633793cf
      Timothy Andrew authored 
      - Pass a `remember_me` query parameter along with the initial OAuth request, and
  pick this parameter up during the omniauth callback from
  request.env['omniauth.params']`.

- For 2FA-based login, copy the `remember_me` param from `omniauth.params` to
  `params`, which the 2FA process will pick up.

- For non-2FA-based login, simply call the `remember_me` devise method to set
  the session cookie.
      633793cf
  3. 03 Jul, 2017 13 commits
  5. 02 Jul, 2017 4 commits
  7. 30 Jun, 2017 19 commits
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7