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

Revoked identity listed as valid #85

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Conversation

sapk
Copy link

@sapk sapk commented Feb 12, 2020

This isn't really a PR but we have have an issue on go-gitea/gitea#6778 with a revoked identity being listed as valid.
The goal is to find the reason and fix this in the PR.

So, I added a test with the user public key to confirm the bug. (If we found the exact reason I will generate a dummy one and rebase)

I haven't found the culprit so if you can have a look.

For the details,
What version of Go are you using (go version)? 1.13
What operating system and processor architecture are you using? linux
What did you do? use a key with revoked id and a valid one
What did you expect to see? only the valid one
What did you see instead? both ids

sidenote: I don't think it is bound to os/platform/go version.

Thanks in advance for any help you can provide.

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

Successfully merging this pull request may close these issues.

1 participant