scandihealth / lpr3-docs

https://scandihealth.github.io/lpr3-docs/
MIT License
11 stars 7 forks source link

ITI41 Response different #302

Closed Remcovanderwerf closed 5 years ago

Remcovanderwerf commented 5 years ago

Hi,

just recently i discovered that the ITI-41 response message is a bit different than before.

before, the element <ns10:FlowStatus>flow_finalized_succesfully</ns10:FlowStatus> was present under the <soap:header>. See: with.txt

Just recently this element is not present anymore. See: without.txt

I used this element in the implementation to validate if the response i received is a valid response of the registry or not (for example "service unavailable" does not hold this element).

Is there a reason that this element is not present anymore? Is there a chance to bring back this element?

Looking forward hearing from you!

Kind regards,

Remco van der Werf Enovation | The Netherlands Contractor: Region Syddanmark

TueCN commented 5 years ago

No this is not an intentional change, it is probably related to the changes in #240

I will look into it

TueCN commented 5 years ago

I have identified the issue and will restore the old behaviour:

Remcovanderwerf commented 5 years ago

Sounds great!

When will this change be active?

TueCN commented 5 years ago

I am testing a fix now. Is the issue blocking for your use of the service? I can deploy it straight away or it can be included in the normal release cycle (Tuesday)

Remcovanderwerf commented 5 years ago

Currently not so the normal deployment on tuesday will suffice.

I will plan on changing my configuration back for wednesdays.

Thank you!

TueCN commented 5 years ago

@Remcovanderwerf we have just deployed a new version (that also includes this fix). Please test now.

Remcovanderwerf commented 5 years ago

Hi @TueCN,

The response is as expected again including the "flow_finalized_succesfully".

Thank you for the investigation and solution, closing this issue.