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

Already on GitHub? Sign in to your account

Opaque field ignored #5

Open
samizdatco opened this Issue Nov 4, 2011 · 1 comment

Comments

Projects
None yet
2 participants
Owner

samizdatco commented Nov 4, 2011

the opaque field is not used when generating challenges, nor is it validated when included in an authentication request. is this a significant omission? the spec makes it seem as though it only exists as a convenience to stash state in, but i could believe some software out there depends upon it...

@erikdubbelboer erikdubbelboer added a commit that referenced this issue Apr 8, 2017

@erikdubbelboer erikdubbelboer Merge pull request #5 from lluuaapp/fix-crash
Fixed crash when authenticating with current macOS/Safari
cc61b4a
Collaborator

erikdubbelboer commented Apr 8, 2017

See

- the opaque field is not used when generating challenges, nor is it validated when included
in an authentication request. is this a significant omission? the spec makes it seem as
though it only exists as a convenience to stash state in, but i could believe some software
out there depends upon it...

@erikdubbelboer erikdubbelboer reopened this Apr 8, 2017

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