Default connection_id to None to account for Connectionless Proofs

[popkinj]

This is a small change in response to this issue and conversation.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[dbluhm]

The formatter can be a pain 😅 thanks for the contribution! Tests look good!

[dbluhm]

@swcurran is there someone on the BC Gov team that has looked at connection-less proofs recently that could spare a minute to check these changes?

If not, I'm inclined to go ahead and merge.

[popkinj]

@swcurran is there someone on the BC Gov team that has looked at connection-less proofs recently that could spare a minute to check these changes?

If not, I'm inclined to go ahead and merge.

Maybe @esune or @andrewwhitehead would like to have a peak. 👀

[esune]

@swcurran is there someone on the BC Gov team that has looked at connection-less proofs recently that could spare a minute to check these changes?

If not, I'm inclined to go ahead and merge.

@popkinj is on the BC Gov team, we triggered this change as we need to receive a problem report for a connection-less proof to handle proof rejection (for VC-AuthN specifically in our case). We are happy to merge!