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

Export users fails where a value is specified in AlternateEmails, resulting in "Please login to LCS again. Your cookie probably expired." error. #98

Closed
bn-cap-01 opened this issue Jul 18, 2024 · 1 comment · Fixed by #99
Closed

Export users fails where a value is specified in AlternateEmails, resulting in "Please login to LCS again. Your cookie probably expired." error. #98

bn-cap-01 opened this issue Jul 18, 2024 · 1 comment · Fixed by #99
Assignees
@bn-cap-01
Labels
bug Something isn't working

Comments

@bn-cap-01
Copy link
Contributor

An exception is thrown on the JSON deserialise, and is caught in the generic exception handling.

The exception is: Error converting value to type 'System.Collections.Generic.List`1[System.String]'. Path 'Results[5].UserProfile.AlternateEmails'
bn-cap-01 added a commit to bn-cap-01/2LCS that referenced this issue Jul 18, 2024
@bn-cap-01
Fix object type for AlternateEmails attribute, issue microsoft#98
43855c1 
Update JsonObjects.cs
@bn-cap-01 bn-cap-01 mentioned this issue Jul 18, 2024
Merged
FH-Inway pushed a commit that referenced this issue Jul 19, 2024
@bn-cap-01 @FH-Inway
Fix object type for AlternateEmails attribute, issue #98
5696d74 
Update JsonObjects.cs
@FH-Inway FH-Inway added the bug Something isn't working label Jul 19, 2024
@FH-Inway
Copy link
Collaborator

Welcome to the repo and thanks for taking the time to register with GitHub to report this issue AND providing a fix as well. That's awesome! ❤️
I noticed the issue as well some time ago, but when I tried to repro it, I could not find a project where it occurs. Found one today though, so I was able to verify the fix works on my side as well.
Released in new version 0.51.0 🚀

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

@bn-cap-01 bn-cap-01

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix object type for AlternateEmails attribute, issue #98 bn-cap-01/2LCS
2 participants
@FH-Inway @bn-cap-01