Skip to content

Enable outbound transport in --no-transport mode, only disable inbound#4050

Merged
jamshale merged 6 commits intoopenwallet-foundation:mainfrom
OpSecId:fix-no-webhooks-on-no-transport
Feb 17, 2026
Merged

Enable outbound transport in --no-transport mode, only disable inbound#4050
jamshale merged 6 commits intoopenwallet-foundation:mainfrom
OpSecId:fix-no-webhooks-on-no-transport

Conversation

@PatStLouis
Copy link
Contributor

Thanks @timbl-ont for raising this issue. The --no-transport mode was added a while back to disable the need to configure outbound and inbound transport if a deployment didn't need to setup didcomm channels. By disabling oubound transport, webhooks were no longer being sent. This PR keeps the oubound transport enabled for http messages.

@PatStLouis PatStLouis marked this pull request as ready for review February 17, 2026 02:10
PatStLouis and others added 3 commits February 16, 2026 21:15
Signed-off-by: Patrick St-Louis <patrick.st-louis@opsecid.ca>
@PatStLouis PatStLouis requested review from ff137 and swcurran February 17, 2026 15:03
Copy link
Member

@ff137 ff137 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jamshale
Copy link
Contributor

@PatStLouis Unable to update the branch with main. Could you do this and we'll get it in.

@sonarqubecloud
Copy link

@jamshale jamshale merged commit 637bc7a into openwallet-foundation:main Feb 17, 2026
12 checks passed
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.

3 participants