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

#129 Expose product name/version from FinTs constructor #130

Closed
wants to merge 2 commits into from
Closed

#129 Expose product name/version from FinTs constructor #130

wants to merge 2 commits into from

Conversation

Philipp91
Copy link
Contributor

This closes #129 and replaces #97. See https://www.hbci-zka.de/register/prod_register.htm for context.

This is a breaking change and needs a major version bump, plus a hint to users that they need to register and pass two more parameters to the constructor.

This is part of the library's main interface, so most users will need its documentation.
As of August 2019, all clients are required to provide these fields, and by design they should identify the end-user product rather than the fints-hbci-php library itself.
@qpwr
Copy link

qpwr commented Oct 6, 2019

Does this work for you, @Philipp91? I included your code into my local testing instance and added the product name in the constructor, but it doesn't seem to make any difference. Unfortunately I didn't find any information about how to enable debugging output... :-(

@Philipp91
Copy link
Contributor Author

This repository is dead, see here: https://github.com/nemiah/phpFinTS

@qpwr
Copy link

qpwr commented Oct 7, 2019

Thanks a lot. Unfortunately it seems that my Sparkasse does no longer support non-PSD2 requests anyway, so it's a dead end for me. :-( But still, thanks for pointing me in the right direction!

@Philipp91 Philipp91 closed this Nov 10, 2019
@Philipp91 Philipp91 deleted the productname branch November 10, 2019 19:20
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.

FinTS registration data
2 participants