UserID is not the relevant value here, it’s the public key (identityKey)

[sirdeggen]

Check identityKey when checking certs

Description of Changes

During Certificate Acquisition we don't need to validate UserID, which is just an internal auto-incrementing ID number; rather we ought to be checking the identityKey which is a universally applicable pubKey.

Linked Issues / Tickets

Unsure, but this fixes:

Running this example code against a go-wallet-toolbox instance locally.

Testing Procedure

I ran the example code, once with no changes which failed:
RPC Error: access is denied due to an authorization error

running after making these changes results in successful issuance.

• I have added new unit tests
• All tests pass locally
• I have tested manually in my local environment

Checklist:

• I have performed a self-review of my own code
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have run the linter

[github-actions]

Manual Tests

ℹ️ Remember to ask team members to perform manual tests and to assign tested label after testing.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud