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

Using UnstructuredRemittanceIdentifier in MD5FingerPrint #7

Merged

Conversation

bountin
Copy link
Contributor

@bountin bountin commented Apr 22, 2015

The EPS API requires the use of the unstructured remittance identifier if it is specified in the XML. The Pflichtenheft is ambiguous in that point but may be fixed in the future. This fact was explained to me by their support team.

Without this patch no transactions with an UnstructuredRemittanceIdentifier can be made as it will return a fingerprint issue (and finally lock your account).

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 96.96% when pulling 75f511b on bountin:bugfix_unstructuredRemittanceIdentifier into 7a4f3fd on hakito:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 96.96% when pulling 75f511b on bountin:bugfix_unstructuredRemittanceIdentifier into 7a4f3fd on hakito:master.

This detail is not clearly described in the current EPS documentation.
@bountin bountin force-pushed the bugfix_unstructuredRemittanceIdentifier branch from 75f511b to 01e2b23 Compare April 22, 2015 14:17
@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 96.96% when pulling 01e2b23 on bountin:bugfix_unstructuredRemittanceIdentifier into 7a4f3fd on hakito:master.

@hakito
Copy link
Owner

hakito commented Apr 23, 2015

Thanks for your contribution. As my partner bank didn't provide any test system and I was using only the RemittanceIdentifier - I never had the problem.

hakito added a commit that referenced this pull request Apr 23, 2015
…ifier

Using UnstructuredRemittanceIdentifier in MD5FingerPrint
@hakito hakito merged commit fcb363a into hakito:master Apr 23, 2015
@bountin
Copy link
Contributor Author

bountin commented Apr 23, 2015

Yeah, the Erste cooperated pretty nicely (especially for a Bank's IT) :)

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