Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

If person token is deleted by another service / manually, laji.fi enters an erroneous state #667

Open
Blodir opened this issue Nov 2, 2024 · 1 comment
Open

If person token is deleted by another service / manually, laji.fi enters an erroneous state #667

Blodir opened this issue Nov 2, 2024 · 1 comment
Assignees
@esko-piirainen @Blodir

Comments

@Blodir
Copy link
Contributor

Blodir commented Nov 2, 2024 •
edited
Loading

Repro:

  1. Log in to dev.laji.fi
  2. Grab the person token from the network tab/localstorage
  3. Send DELETE to dev.laji.fi/api/person-token/
  4. Refresh page, then try to open your profile in dev.laji.fi

This came up when testing logout in laji-dmp frontend.
@Blodir Blodir added bug Something isn't working important This task must be done labels Nov 2, 2024
@Blodir Blodir self-assigned this Nov 2, 2024
@Blodir Blodir changed the title If person token is deleted by another service / manually, laji.fi does not check that the token is invalid If person token is deleted by another service / manually, laji.fi enters an erroneous state Nov 2, 2024
@Blodir
Copy link
Contributor Author

Blodir commented Nov 2, 2024

May be an API issue actually, I got to a state where /person/<id> endpoint responded with the same person for two different tokens 🤔 And the first of the tokens can't be deleted (since it has been already deleted)... Requires investigation

@wkmor1 wkmor1 removed the bug Something isn't working label Nov 4, 2024
@olzraiti olzraiti assigned esko-piirainen and unassigned olzraiti Nov 4, 2024
@wkmor1 wkmor1 added this to the Release 78 milestone Nov 4, 2024
@Blodir Blodir removed the important This task must be done label Nov 6, 2024
@wkmor1 wkmor1 removed this from the Release 78 milestone Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@esko-piirainen esko-piirainen

@Blodir Blodir

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@esko-piirainen @wkmor1 @olzraiti @Blodir