-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add more non-problematic strings to ResultAnalyzer #1
Comments
Will solve first part of Issue Mestrona#1
Nowadays I also get the message: Selecting PAIN format... Which throws an error. Not sure if anyone else is experiencing this. |
Which AqBanking version are you using? Might be a new message and seems to be save to ignore for me. |
That's very well possible: Versions: Full message is sth. like this: |
I also sometimes see this error message:
This one is not save to exclude I guess. However the official mailing list could not help. Did you see this one as well? |
Hmm, had a look into the jobtransferbase.c it seems that the log level of "Selecting PAIN format" is error. I guess I need to ask the author if this is correct.. |
I think the loglevels are not very granular in AqBanking, I would be best
du have a --quiet switch and we could avoid the filtering completely.
|
There is
see https://github.com/aqbanking/aqbanking maybe we should set this env to "error" or even "critical" when calling CC |
It is already "error". aqbanking-php/src/Command/ShellCommandExecutor.php Lines 9 to 15 in 1fdb337
No clue on this, can't provoke messages to see any difference in output from level error upwards. |
In src/AqBanking/Command/ShellCommandExecutor/ResultAnalyzer.php
This should be added:
This might as well be needed on some setups - it does not seem to be critical - but there might be problems with Unicode Characters. Maybe we need a strict- and not-so-strict-mode
The text was updated successfully, but these errors were encountered: