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

Fix specific hashing options #95

Closed
iegomez opened this issue Sep 22, 2020 · 0 comments · Fixed by #102
Closed

Fix specific hashing options #95

iegomez opened this issue Sep 22, 2020 · 0 comments · Fixed by #102
Assignees
Labels

Comments

@iegomez
Copy link
Owner

iegomez commented Sep 22, 2020

When building a hasher we may pass a backend to check for specific options. For Postgres, docs say the prefix should be pg, just as in other Postgres related options, but here we are passing postgres instead, resulting in specific hasher options being discarded: https://github.com/iegomez/mosquitto-go-auth/blob/master/go-auth.go#L257
This may be true for other backends too.

The solution would be to have a backends map name => prefix and derive the correct prefix to pass to the hasher from it.

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

Successfully merging a pull request may close this issue.

1 participant