Create a new version of the Pub / Sub connector

GoutsmitSam commented 1 year ago

The Pub/Sub connector has some issues

using an (very) old version of Web API development
using deprecated Azure packages with vulnerabilities.
using not so secure Basic Authentication

However, the packages cannot easily be upgraded. Reason: the pubsub library was never meant to be used via an API. If a listener reads a message from SB it caches it on that instance, if via an API call you try to complete a message and it is not on the same instance as the listener it fails (it's in-memory stateful).

Best to rework them.

stijnmoreels commented 1 year ago

Great that we have this, but wondering if this is really an 'enhancement' and not an issue/bug.

GoutsmitSam commented 8 months ago

This is currently under investigation, as it will become necessary short term to upgrade the Pub/Sub to .NET Standard. Reasons:

Proximus requirement that all Invictus elements be able to run on Linux
Broader: Roadmap requirement to be able to run Invictus in Container Apps

First step:

Proof of Concept to confirm / deny that the problem is still present

invictus-integration / docs-ifa

Create a new version of the Pub / Sub connector #179