Skip to content

fix: bundesbank url doesn't work with http anymore, updated to https#2

Open
XPanniX wants to merge 1 commit intobav-php:masterfrom
XPanniX:master
Open

fix: bundesbank url doesn't work with http anymore, updated to https#2
XPanniX wants to merge 1 commit intobav-php:masterfrom
XPanniX:master

Conversation

@XPanniX
Copy link

@XPanniX XPanniX commented Mar 29, 2018

When auto-updating the bundesbank file an error was thrown:
PHP Fatal error: Uncaught malkusch\\bav\\DataBackendIOException: Failed to download 'http://www.bundesbank.de/Redaktion/DE/Standardartikel/Aufgaben/Unbarer_Zahlungsverkehr/bankleitzahlen_download.html'. in /var/www/iban-test/vendor/malkusch/bav/classes/dataBackend/file/download/Downloader.php:73\nStack trace:\n#0 /var/www/iban-test/vendor/malkusch/bav/classes/dataBackend/file/FileDataBackend.php(162): malkusch\\bav\\Downloader->downloadContent('http://www.bund...')\n#1 /var/www/iban-test/vendor/malkusch/bav/classes/dataBackend/update/AutomaticUpdatePlan.php(49): malkusch\\bav\\FileDataBackend->update()\n#2 [internal function]: malkusch\\bav\\AutomaticUpdatePlan->malkusch\\bav\\{closure}()\n#3 /var/www/iban-test/vendor/malkusch/bav/classes/util/Lock.php(88): call_user_func(Object(Closure))\n#4 /var/www/iban-test/vendor/malkusch/bav/classes/dataBackend/update/AutomaticUpdatePlan.php(54): malkusch\\bav\\Lock->nonblockingExecuteOnce(Object(Closure))\n#5 /var/www/iban-test/vendor/malkusch/bav/classes/dataBackend/DataBackendContainer.php(72): malkusc in /var/www/iban-test/vendor/malkusch/bav/classes/dataBackend/file/download/Downloader.php on line 73

The (old) http://-URL of the bundesbank redirects to https:// which cURL somehow didn't follow when downloading the bankleitzahlen-file. In order to fix that I changed the bundesbank's URL to https://.

asdfklgash pushed a commit to asdfklgash/bav that referenced this pull request Apr 27, 2023
- Fixing the error: PHP Fatal error:  Uncaught ValueError: trigger_error(): Argument bav-php#2 ($error_level) must be one of E_USER_ERROR, E_USER_WARNING, E_USER_NOTICE, or E_USER_DEPRECATED
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.

2 participants

Comments