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

User ID type INT screws up postgres #1825

Closed
plettich opened this issue Aug 27, 2019 · 0 comments · Fixed by #1829

Comments

@plettich
Copy link
Member

commented Aug 27, 2019

If the UserIDResolver returns a UID of type int, the resulting query on the TokenOwner table (where the UID is of type varchar) messes up postgres.
Since we use a UID of type string, the getUserId() function should return only strings.

@plettich plettich added this to the 3.2 milestone Aug 27, 2019

@plettich plettich self-assigned this Aug 27, 2019

@plettich plettich added this to To do in privacyIDEA 3.2 via automation Aug 27, 2019

plettich added a commit that referenced this issue Aug 27, 2019
The getUserId() function returns only strings
privacyIDEA works with a user ID type string so we enforce that the
`getUserId()`-function only returns strings.

Fixes #1825

@plettich plettich moved this from To do to In progress in privacyIDEA 3.2 Aug 27, 2019

plettich added a commit that referenced this issue Aug 28, 2019
The getUserId() function returns only strings
privacyIDEA works with a user ID type string so we enforce that the
`getUserId()`-function only returns strings.

Fixes #1825
Also fixes #862 (yay!)
plettich added a commit that referenced this issue Aug 28, 2019
The getUserId() function returns only strings
privacyIDEA works with a user ID type string so we enforce that the
`getUserId()`-function only returns strings.

Fixes #1825
Also fixes #862 (yay!)

privacyIDEA 3.2 automation moved this from In progress to Done Aug 29, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
1 participant
You can’t perform that action at this time.