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.

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

Screen name change not updating blocklyprop user entry #1649

Open
zfi opened this issue Feb 12, 2019 · 2 comments
Open

Screen name change not updating blocklyprop user entry #1649

zfi opened this issue Feb 12, 2019 · 2 comments

Comments

@zfi
Copy link
Contributor

zfi commented Feb 12, 2019

Changing a screen name only changes the screen name in the user's cloudsession record. It needs to propogate to the user's blocklyprop record.

@zfi zfi added the bug label Feb 12, 2019
@zfi zfi self-assigned this Feb 12, 2019
@zfi zfi added this to ToDo in 2019 - February Update via automation Feb 13, 2019
@zfi
Copy link
Contributor Author

zfi commented Feb 13, 2019

This has major implications for locating projects owned by a user. The projects will go missing if the screen name in the blocklyprop user table does not match the screen name in the user's profile.

@zfi
Copy link
Contributor Author

zfi commented Feb 20, 2019

Session data for an anonymous user looks like this in the application log:

SessionData{user=null, idUser=null, locale=null}

and this is what an authenticated user looks like in the application log:

INFO [UserServiceImpl.java:198] SessionData SessionData{user=com.parallax.client.cloudsession.objects.User@5249309c6b, idUser=1, locale=en_US}

It now appears that the login process was not setting this data correctly. Post login, the attributes were still null values.

The code has been updated to accomplish two aligned goals. The first is that a change in the screen name in the user profile is now propagated to both databases to ensure that they remain synchronized. Additionally, when processing a login, a check is made to ensure that both databases agree on the user's screen name and update them if they do not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

1 participant