Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • D djangoldp-account
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 20
    • Issues 20
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 1
    • Merge requests 1
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • djangoldp-packages
  • djangoldp-account
  • Issues
  • #79

Closed
Open
Created Nov 05, 2021 by Calum Mackervoy@calummackervoyMaintainer

default_redirect_uri can become invalid

Myself and @plup fixed a bug just now with a user who had a default_redirect_uri which was taking them back to the OIDC authorize view with an old (now invalid) client_id. So they successfully login, and they're taken back to an invalid screen

There was a similar issue where an error string was included in the redirect URI and this led people

The default_redirect_uri was included on the user model as a mechanism to know where to redirect someone when none was passed in with the request (especially on federated login - the idea was that I visit one client application, I'm redirected to my identity provider, but the client didn't pass in that request where to redirect them back, so it looks at the last lodged in app stored as default_redirect_uri. Clearly if this has been set to an error URI then it will look like I've hit an error!)

Assignee
Assign to
Time tracking