Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IIA-APPROVAL-RESPONSE contain V6 condition hash instead new V7 iiaHash #174

Open
skishk opened this issue Apr 23, 2024 · 9 comments
Open

Comments

@skishk
Copy link

skishk commented Apr 23, 2024

Dear Colleagues,

is it possibile to use conditionHash code of the old IIA in V6 as the new iiaHash in the IIA-APPROVAL-RESPONSE ?

@janinamincer-daszkiewicz
Copy link
Member

No:

<xs:element name="iia-hash" type="ewp:Sha256Hex" minOccurs="1" maxOccurs="1">

@demilatof
Copy link

No:

<xs:element name="iia-hash" type="ewp:Sha256Hex" minOccurs="1" maxOccurs="1">

What could be the consequence for the providers that make this kind of error and for their partner?
I suppose that an approved IIA would seem as not approved.

@janinamincer-daszkiewicz
Copy link
Member

Yes. Also, because conditionHash is not based on XSLT you cannot verify XSLT v6 against XSLT v7. This will trigger the new approval process.

@skishk
Copy link
Author

skishk commented Apr 23, 2024

No:

<xs:element name="iia-hash" type="ewp:Sha256Hex" minOccurs="1" maxOccurs="1">

I opened a tkt one week ago ESCI-11994 , but the problem is still in production...

@janinamincer-daszkiewicz
Copy link
Member

I will try to intervene.

@skishk
Copy link
Author

skishk commented Apr 23, 2024

No:

<xs:element name="iia-hash" type="ewp:Sha256Hex" minOccurs="1" maxOccurs="1">

What could be the consequence for the providers that make this kind of error and for their partner? I suppose that an approved IIA would seem as not approved.

consequence is not really nice for who is working with hundreds of IIAs...(IROs) it's like everything with V7 is no more approved....
we are trying to contain this problem as is our possibilities... but we hope they will resolve quickly the problem... obviously this require extra effort for all of us... (IT and IROs) 😰

@skishk
Copy link
Author

skishk commented Apr 23, 2024

I will try to intervene.

thank you very much

@demilatof
Copy link

demilatof commented Apr 23, 2024

Yes. Also, because conditionHash is not based on XSLT you cannot verify XSLT v6 against XSLT v7. This will trigger the new approval process.

Well, I agree.
I don't need to test this aspect, because we will enter into production straightly with version 7, but the providers aren't tested to verify that they support the migration from v6 to v7?

@skishk
Copy link
Author

skishk commented Apr 23, 2024

Yes. Also, because conditionHash is not based on XSLT you cannot verify XSLT v6 against XSLT v7. This will trigger the new approval process.

Well, I agree. I don't need to test this aspect, because We will enter into production straightly with version 7, but the providers aren't tested to verify that they support the migration from v6 to v7?

No… :(

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

No branches or pull requests

3 participants