openwallet-foundation / acapy

ACA-Py is a foundation for building decentralized identity applications and services running in non-mobile environments.
https://aca-py.org
Apache License 2.0
419 stars 512 forks source link

"No endorser invitation, can't connect automatically." message on startup even when the agent instance is connected to an endorser service #3308

Closed WadeBarnes closed 4 weeks ago

WadeBarnes commented 4 weeks ago

Observed behavior:

When upgrading an agent instance (0.10.2 to 0.12.2) in this case I started seeing the following message:

2024-10-23 13:10:34,142 aries_cloudagent.utils.endorsement_setup INFO No endorser invitation, can't connect automatically.

The agent in question was already connected to an endorser service and the connection was know to be working.

Expected behavior:

To not see this message when an agent already has a working connection established with as endorser service. This message should also not be displayed if (these scenarios were not tested):

Steps to reproduce:

This issue is not restricted to upgrades. The same issue will occur if you setup a new agent instance with an endorser connection.

jamshale commented 4 weeks ago

I think i screwed this up :/

But, I'm pretty sure the error message is misleading. The previous endorser connection should still exist and work correctly.

I'll clean up the process for when there is an existing endorser connection. I don't know why I made it exit with that message.

WadeBarnes commented 4 weeks ago

You are connect, from what I can tell the existing endorser connection is not affected and still works, it's existence is just not being taken into account.