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

Inconsistency in verifying account ownership with 4 different credentials #117

Open
mh-concordium opened this issue Sep 11, 2023 · 1 comment
Labels
[Project] Concordia [Type] Change Request Some visible functionality should be change.

Comments

@mh-concordium
Copy link

Bug Description
This is a case where I have issued credentials twice, so I have 2 active credentials for Telegram (T1 and T2) and 2 active credentials for Discord (D1 and D2). It seems that there is inconsistent behavior in how it is possible to prove ownership over the accounts. Using credentials pairs to prove as described in the following steps below.

Steps to Reproduce

  1. Issue 2 credentials for Telegram (T1 and T2) and 2 credentials for Discord (D1 and D2).
  2. Verify with T1 and D1 -> Proof OK - Works as expected
  3. Don't remove verification.
  4. Verify with T2 and D2 -> Proof rejected - works as expected.
  5. Verify with T1 and D2 -> Proof OK - works as expected.
  6. Verify with T2 and D1 -> Proof rejected - not working as expected.

Expected Result
It should be possible to verify ownership over the accounts in step 6. That would be consistent with verification in step 5.

Actual Result
With the step 6, verification is not successful.

Versions

  • Software Version: Concordia 1.0.0
@mh-concordium mh-concordium added the [Type] Bug Something isn't working label Sep 11, 2023
@mh-concordium
Copy link
Author

With the current implementation this is expected behavior since at the step 6, the credentials that were used are both different than the ones that were used at step 5. In a sense, it is the same as proof rejected at step 4 where you are basically trying to prove the same thing twice.

@mh-concordium mh-concordium added [Type] Change Request Some visible functionality should be change. and removed [Type] Bug Something isn't working labels Sep 11, 2023
@soerenbf soerenbf changed the title Inconsistency in verifying account ownership with 4 different credentials Inconsistency in verifying account ownership with 4 different credentials Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Project] Concordia [Type] Change Request Some visible functionality should be change.
Projects
None yet
Development

No branches or pull requests

2 participants